ASP.NET Identity is referencing CoreWiki.Core directly
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 25/100
- issue の種類
- リファクタリング
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- csharp
調査の方向性
まず、CoreWikiUser を参照している Identity コードを追跡し、IUserStore のカスタムストレージプロバイダーに関するドキュメントを確認します。次に、UserManager とセッション処理を現在のレイヤーに置くべきか、それともアプリケーション抽象化に置くべきかを判断します。Identity の実装が CoreWiki.Core に直接依存しなくなり、関連するテストによって分離が検証されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
The code of ASP.NET Identity is using the domain object CoreWikiUser directly. There is also no abstraction of the Identity code, making it impossible to refactor it out to the Application layer.
The issue has been raised before in the Identity repo: https://github.com/aspnet/Identity/issues/883
As that issue states it has been fixed, maybe we need to look what consequences that has for our code and how we can improve it to achieve that separation.
The documentation can help there as well as it show how to create a IUserStore: https://docs.microsoft.com/en-us/aspnet/core/security/authentication/identity-custom-storage-providers?view=aspnetcore-2.1
That takes at least the EF out of the equation. UserManager is going to be trickier, not sure yet whether that is needed. It depends on whether the session lives at the front-end or the back-end. If it's front-end only, then the mobile/API back-end (once created) might need its own depending on the functionality being exposed.
- 主要言語
- CSS
- スター
- 438
- フォーク
- 159
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
csharpfritz/CoreWiki のほかの issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 35/100
csharpfritz/CoreWiki#463 · コメント 1 件 · リアクション 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
csharpfritz/CoreWiki#406 · コメント 1 件 · リアクション 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
csharpfritz/CoreWiki#353 · コメント 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
csharpfritz/CoreWiki#351 · コメント 1 件 ·
-
enhancement help wanted
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
csharpfritz/CoreWiki#349 ·
csharpfritz/CoreWiki の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
OpenPrinting/cups#1772 ·
メンテナーはふだん 1 日以内に返信
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
scarif-labs/horus#61 ·
メンテナーはふだん 1 日以内に返信
-
agent/sec-check help wanted security stage/010-actionable
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
hivecommons/hotshot#235 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
registrystack/registry-stack#2168 ·
メンテナーはふだん 1 日以内に返信
-
Medium Priority Security vulnerabilities
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
IntersectMBO/developer-experience#339 ·
メンテナーはふだん 2 日以内に返信