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

Vue Jwt #2151

Closed
wants to merge 3 commits into from
Closed

Vue Jwt #2151

wants to merge 3 commits into from

Conversation

matthieuRioual
Copy link
Contributor

@matthieuRioual matthieuRioual commented Jun 19, 2022

Add authentication progress to vue application through pinia storage.

A login page has been created as a guard route.

Once connected you can access application single page and check your identity or ever log out.

Before adding jwt to vue generated application, make sur you added pinia or it gonna throw an error.

You can test it on this branch with credentials "admin, admin"

fix #897

@matthieuRioual matthieuRioual marked this pull request as draft June 19, 2022 00:39
@codecov
Copy link

codecov bot commented Jun 19, 2022

Codecov Report

Merging #2151 (4818f82) into main (311095d) will decrease coverage by 0.02%.
The diff coverage is 98.37%.

❗ Current head 4818f82 differs from pull request most recent head 040b289. Consider uploading reports for the commit 040b289 to get more accurate results

@@              Coverage Diff              @@
##                main    #2151      +/-   ##
=============================================
- Coverage     100.00%   99.97%   -0.03%     
- Complexity      2405     2447      +42     
=============================================
  Files            504      522      +18     
  Lines           8666     9022     +356     
  Branches         189      204      +15     
=============================================
+ Hits            8666     9020     +354     
- Misses             0        2       +2     
Impacted Files Coverage Δ
...nerator/client/vue/security/jwt/domain/VueJwt.java 86.66% <86.66%> (ø)
...rity/jwt/application/VueJwtApplicationService.java 100.00% <100.00%> (ø)
...t/vue/security/jwt/domain/VueJwtDomainService.java 100.00% <100.00%> (ø)
...infrastructure/config/VueJwtBeanConfiguration.java 100.00% <100.00%> (ø)
...wt/infrastructure/primary/rest/VueJwtResource.java 100.00% <100.00%> (ø)
src/main/webapp/app/common/domain/Service.ts 100.00% <0.00%> (ø)
.../main/webapp/app/common/secondary/RestServiceId.ts 100.00% <0.00%> (ø)
...p/app/springboot/secondary/SpringBootRepository.ts 100.00% <0.00%> (ø)
...p/springboot/secondary/client/AngularRepository.ts 100.00% <0.00%> (ø)
.../springboot/primary/generator/ServiceProjection.ts 100.00% <0.00%> (ø)
... and 62 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 311095d...040b289. Read the comment docs.

@pascalgrimaud
Copy link
Member

With the new module system, it should be easier now
Do you want to give a new try @matthieuRioual ?

@matthieuRioual
Copy link
Contributor Author

matthieuRioual commented Jul 23, 2022 via email

@pascalgrimaud
Copy link
Member

Let's close this one, as it will be more work to rebase it
it's better to start the work from scratch using the module system

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vue: account with authentication JWT
2 participants