Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JUBE-2-Create-Authentication-Page #48

Merged
merged 1 commit into from
Aug 4, 2024

Conversation

richard-churchman
Copy link
Contributor

Moved logic in the Authentication Controller to a dedicated service so that it can be reused by the Blazor user interface. Moved validations for the Authentication processing into a dedicated class, for reuse also. Ported the authentication page to Blazor using more modern design patterns, relying on DI and declarations where possible. Some modifications to Dto's affecting the Authentication Controller in Jube.App (current version). Implemented resource localisation in the Blazor user interface, although its implementation in Validations is pending. Implemented a declarative policy authentication for pages, although intend to explore ways to reference the claim directly in the Authorize attribute (e.g. [Authorize Claim(Permission,1)]) rather than relying on Policy, which depends on claims anyway. Some typo changes and nullable field refactoring implemented.

…o that it can be reused by the Blazor user interface. Moved validations for the Authentication processing into a dedicated class, for reuse also. Ported the authentication page to Blazor using more modern design patterns, relying on DI and declarations where possible. Some modifications to Dto's affecting the Authentication Controller in Jube.App (current version). Implemented resource localisation in the Blazor user interface, although its implementation in Validations is pending. Implemented a declarative policy authentication for pages, although intend to explore ways to reference the claim directly in the Authorize attribute (e.g. [Authorize Claim(Permission,1)]) rather than relying on Policy, which depends on claims anyway. Some typo changes and nullable field refactoring implemented.
@richard-churchman richard-churchman merged commit 69d8d22 into master Aug 4, 2024
3 checks passed
@richard-churchman richard-churchman deleted the JUBE-2-Create-Authentication-Page branch August 4, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant