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

Upgrade JHipster microservices to use Spring Security 5.1 OIDC #1

Open
wants to merge 17 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@mraible
Copy link
Owner

commented Feb 27, 2019

Current status: you can log into the gateway, but calls to microservices do not work. It seems routes are not getting registered with the registry properly.

I tried upgrading the JHipster Registry to fix this but failed.

@mraible mraible changed the title Upgrade JHipster Microservices to use Spring Security 5.1 OIDC Upgrade JHipster microservices to use Spring Security 5.1 OIDC Feb 27, 2019

@mraible mraible marked this pull request as ready for review Mar 9, 2019

@mraible

This comment has been minimized.

Copy link
Owner Author

commented Mar 10, 2019

Update: registering routes now works, but requests to the downstream "blog" microservice do not.

@ruddell

This comment has been minimized.

Copy link

commented Mar 10, 2019

@mraible It sounds like you're experiencing this issue: jhipster/generator-jhipster#9261

It's due to LazyLoading the zuulFilterInitializer, I haven't had time to look into the correct fix.

@mraible

This comment has been minimized.

Copy link
Owner Author

commented Mar 10, 2019

@ruddell That was it! You're a genius, man!

I changed LazyInitBeanFactoryPostProcessor to have the following and everything is working now.

private Class<?>[] exclusionList = {ZuulFilterInitializer.class};

@PierreBesson PierreBesson referenced this pull request Mar 11, 2019

Closed

Microservice APIs returning empty response (master) #9261

1 of 1 task complete

@mraible mraible referenced this pull request Mar 11, 2019

Closed

Upgrade Spring Security's OIDC Support #9276

4 of 4 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.