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

Fix for gh-226 #227

Merged
merged 2 commits into from
Sep 1, 2024
Merged

Fix for gh-226 #227

merged 2 commits into from
Sep 1, 2024

Conversation

ebbe-brandstrup
Copy link
Contributor

Also updated some @param javadocs that had gotten out of sync with the actual method signatures and a few typos. Added Angular build output cache folders to gitignore.

Fixes #226.

…ut of sync with the actual method signatures and a few typos. Added Angular build output cache folders to gitignore.
@@ -1,6 +1,7 @@
package com.c4_soft.springaddons.security.oidc.starter.reactive.resourceserver;

import java.nio.charset.Charset;
import java.sql.Date;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we want java.util.Date here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gotcha - good catch. IntelliJ auto import and I should have caught that while committing.

Copy link
Owner

@ch4mpy ch4mpy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for taking the time to investigate and fix.

@ch4mpy ch4mpy merged commit 8e40263 into ch4mpy:master Sep 1, 2024
1 check passed
@ebbe-brandstrup ebbe-brandstrup deleted the gh-226 branch September 2, 2024 07:26
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.

Parsing of token introspection responses from Keycloak fails when using the reactive stack
2 participants