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

feat: refactoring security.guard.ts #1924

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

klopfdreh
Copy link
Contributor

@klopfdreh klopfdreh commented May 22, 2023

Simplify security.guard.ts

@klopfdreh
Copy link
Contributor Author

Hey @oodamien / @claudiahub - can you have a look why this build failed? https://github.com/spring-cloud/spring-cloud-dataflow-ui/actions/runs/5045332345/jobs/9049470879 - as you can see I just added some . in the comments to trigger the rebuild without changing anything else and the build was a success. 😨

@klopfdreh
Copy link
Contributor Author

@markpollack - I would suggest to also include this in 2.11.0 as there were some async calls missing a leading await statement - this can lead to unwanted behaviors.

@ilayaperumalg ilayaperumalg added this to the 3.4.x milestone Dec 11, 2023
@cppwfs cppwfs modified the milestones: 3.4.x, 4.0.x Sep 10, 2024
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