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

java 21 note #3250

Merged
merged 2 commits into from
Sep 5, 2024
Merged

java 21 note #3250

merged 2 commits into from
Sep 5, 2024

Conversation

lyleschemmerling
Copy link
Contributor

There are some other mentions in places like blogs about Java 17, but this is the only real doc doc I saw that needed to change.

@lyleschemmerling lyleschemmerling requested review from a team as code owners September 5, 2024 18:59
@lyleschemmerling lyleschemmerling merged commit f6f9352 into release/1.53.0 Sep 5, 2024
2 checks passed
@lyleschemmerling lyleschemmerling deleted the lyle/ENG-3/java-21-update branch September 5, 2024 19:43
@@ -30,7 +30,13 @@ Docker, k8s and other container platforms are supported because the host operati

FusionAuth ships with its own version of Java. The version that is contained in the bundle is the required and supported version.

Currently, Java 17 is the supported major version. Java 14 is the supported major version for all FusionAuth releases before FusionAuth 1.32.
Currently, Java 21 is the supported major version. The versions of Java that are supported for prior versions of FusionAuth are:
Copy link
Member

Choose a reason for hiding this comment

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

Looks like our runner / vale is broken?
We of course don't want to use fusionauth instead of FusionAuth.

I had a similar issue in my PR for using GitHub and it suggested I change it. 🤷

|--------------------|--------------|
| 1.53.0 and newer | Java 21 |
| 1.32.0 - 1.52.1 | Java 17 |
| 1.31.0 and earlier | Java 11 |
Copy link
Member

Choose a reason for hiding this comment

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

We've never used Java 11 in the field. We went from 8 -> 14 -> 17 -> 21.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's what I meant!

andrewpai added a commit that referenced this pull request Sep 6, 2024
* Webhook Event Log (#3197)

* add webhookEventLogConfiguration to system config
* document webhook_event_log_viewer role
* doc for retrieving webhook event log APIs
* document webhook event log admin UI

* Document new login intent TTL (#3129)

* add doc for new loginIntent TTL
* add doc for fusionauth.li cookie

* add release notes for SSO Zone changes

* adjust known issue comment

* java 21 note (#3250)

* java 21 note

* wording

* oops

* Added additional release notes.

* correct since version for loginIntentTimeToLiveInSeconds

* Added login lambda doc

* 1. Fix Known Issues to follow the existing format.
2. Add missing Docker image removal.
3. Move new lambda function from Changed to New.
4. Move new action buttons from Changed to Enhancements. Changed should be reserved for things that will potentially affect integrations.
5. Normalize "Update dependencies." and "Update 3rd party dependencies."
6. Expand upon the potential impact to the connection pool changes.
7. Correct formatting in release notes for 1.50.0.
8. Use the correct form of BreadCrumbs.

* 1. We don't serialize `null` values in a JSON response, so we should not document `null`, instead we should indicate when a field will be omitted from the response.
2. Be more consistent with the definition of the Id.
3. Delete dead imports (EventTypes and InlineField) in _webhook-attempt-log-response-body-base.mdx.

* 1. Remove duplicate assignment copy.
2. Don't mention the LoginLambdaValidationContext, this is an internal object.
3. context is read only.
4. Mention when identityProvider is found in the context.
5. Fix broken link to error doc.
6. Don't use fully qualified URLS to docs, use relative links.
7. Remove trailing comma on JSON example
7. Discuss localization.

* Remove SCIM, this was actually updated last release.
#3248

* Various copy and formatting edits.

* - Add tenant.lambdaConfiguration.loginValidationId to Tenant API
- Rename LambdaTypeApi -> LambdaTypeAPI
- Describe the behavior differences in optional messages for themes in 1.53.0.

* Add new lambda field to the tenants overview.

* document since version for webhook_event_log_viewer role

* Update astro/src/content/docs/extend/code/lambdas/login-validation.mdx

Co-authored-by: Dan Moore <dan@fusionauth.io>

* Fix formatting for DeprecatedSince to work in APIField and inline usages.

* Small wording changes.

* - Fix formatting with RemovedSince, AvailableSince, BeforeVersion and DeprecatedSince
- Remove old system config reference
- Remove dead imports

* Add Known Issues.

* Added 1.53.1 release notes.

* Shrink images

---------

Co-authored-by: Spencer Witt <spencer@fusionauth.io>
Co-authored-by: Lyle Schemmerling <lshem16@gmail.com>
Co-authored-by: Lyle Schemmerling <lyle.schemmerling@fusionauth.io>
Co-authored-by: Andy Pai <andy@fusionauth.io>
Co-authored-by: Daniel DeGroff <daniel@fusionauth.io>
Co-authored-by: Dan Moore <dan@fusionauth.io>
Co-authored-by: andrewpai <andrewpai@users.noreply.github.com>
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.

3 participants