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

Update websocket integration docs #15438

Merged

Conversation

regiuss-own
Copy link
Contributor

Updating integration websocket documentation
Injecting ApplicationContext and AuthorizationManager is not intuitive
AuthenticatedAuthorizationManager.authenticated() does not accept security settings

@pivotal-cla
Copy link

@regiuss-own Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@regiuss-own Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 18, 2024
@jzheaux jzheaux self-assigned this Sep 18, 2024
@jzheaux jzheaux added in: docs An issue in Documentation or samples type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Sep 18, 2024
@jzheaux jzheaux added this to the 6.4.0-RC1 milestone Sep 18, 2024
@jzheaux jzheaux changed the title update websocket integration docs Update websocket integration docs Sep 18, 2024
@jzheaux jzheaux force-pushed the feature/update-websocket-docs branch from 1583c36 to ecc421a Compare September 18, 2024 16:57
@jzheaux jzheaux merged commit 4a9a350 into spring-projects:main Sep 18, 2024
6 checks passed
@jzheaux
Copy link
Contributor

jzheaux commented Sep 18, 2024

Thanks for the improvement, @regiuss-own! This is now merged into main. You'll be able to see your changes here once the CI completes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants