-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add Spring Boot 3.0 support for SCIMple #104
Conversation
NOTE: The Java 11 CI build does NOT build the Spring-Boot 3 bits. Clone the project, use Java 17 (SDK MAN!), and build Run the project from your IDE or terminal: |
|
||
```bash | ||
# httpie | ||
http :8080/Users |
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.
Should /Users
be capitalized?
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.
Oddly enough, yes: https://www.rfc-editor.org/rfc/rfc7644#section-3.2
For some reason that capitalization seems really odd to me, but it's in the spec 🤷
ea5473a
to
c2c635f
Compare
a855464
to
79435a4
Compare
31e20fa
to
43fa2e3
Compare
When I check out this branch and run
My Java version:
|
@mraible good catch! Luckily M5 was released last week, so we no longer need to worry about depending on a SNAPSHOT version of Spring (so this type of thing won't happen again)! (This should be fixed in my last commit) |
Updated build to require Java 17, but target 11 |
No description provided.