-
Notifications
You must be signed in to change notification settings - Fork 332
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
Fixes auth interceptor on MusicJungle #145
Conversation
…p the flow when user is not logged in;
@BeforeCall | ||
public void intercept() throws InterceptionException { | ||
@AroundCall | ||
public void intercept(SimpleInterceptorStack stack) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weird indent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't want to change it because I thought the problem was with my IDE so I left that way... Don't worry, I'll fix it :-)
So now, @lucascs? |
It's good now =) I'd only check if that |
I tried that, but it didn't work because there's no other interceptor catching ValidatorException. |
Bummer. |
Ok, sir! |
I just add a simple comment. Do you think it's enough? |
Ok. 🚢 |
Can we merge it? |
👍 |
Fixes auth interceptor on MusicJungle
During my tests on music-jungle when I access an URL not permitted I get this error:
When interceptor uses @BeforeCall it can't stop the flow of request when the user isn't logged in, so it's necessary to use @AroundCall instead.