Skip to content

play-java-streaming-example: Add csp nonce#483

Merged
mkurz merged 1 commit intoplayframework:3.0.xfrom
mkurz:fix-play-java-streaming-example
Dec 19, 2023
Merged

play-java-streaming-example: Add csp nonce#483
mkurz merged 1 commit intoplayframework:3.0.xfrom
mkurz:fix-play-java-streaming-example

Conversation

@mkurz
Copy link
Member

@mkurz mkurz commented Dec 12, 2023

For Comet this depends on

which will be part of Play 3.0.1/2.9.1

@mkurz
Copy link
Member Author

mkurz commented Dec 12, 2023

@Mergifyio backport 2.9.x

@mergify
Copy link
Contributor

mergify bot commented Dec 12, 2023

backport 2.9.x

✅ Backports have been created

Details

@mkurz mkurz force-pushed the fix-play-java-streaming-example branch 2 times, most recently from 9ca39dc to acfab0d Compare December 12, 2023 13:56
@mkurz mkurz force-pushed the fix-play-java-streaming-example branch from acfab0d to 746c8b6 Compare December 12, 2023 13:59
@ihostage
Copy link
Member

I tried to fix this in separated branch isn't related with my Gradle branches and you could just force push my branch and don't spend a time for new PR 😉 Anyway, It is LGTM 👍

@mkurz
Copy link
Member Author

mkurz commented Dec 12, 2023

I tried to fix this in separated branch isn't related with my Gradle branches and you could just force push my branch and don't spend a time for new PR

I know, I just started the example locally to see what's going on without your changes and then started to migrate the Scala code to Java and then just fixed the rest like you did, didn't take so much time, don't worry 😉

@mkurz mkurz force-pushed the fix-play-java-streaming-example branch from 746c8b6 to 368ac27 Compare December 19, 2023 14:12
@mkurz mkurz merged commit 51d3943 into playframework:3.0.x Dec 19, 2023
@mkurz mkurz deleted the fix-play-java-streaming-example branch December 19, 2023 14:25
mkurz added a commit that referenced this pull request Dec 19, 2023
[2.9.x] play-java-streaming-example: Add csp nonce (backport #483) by @mkurz
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.

2 participants