Skip to content

[2.8.x] Bugfix: Always set play.server.http.port, even if disabled#12256

Merged
mkurz merged 1 commit into
playframework:2.8.xfrom
mkurz:28x_http.port_disabled
Dec 6, 2023
Merged

[2.8.x] Bugfix: Always set play.server.http.port, even if disabled#12256
mkurz merged 1 commit into
playframework:2.8.xfrom
mkurz:28x_http.port_disabled

Conversation

@mkurz

@mkurz mkurz commented Dec 6, 2023

Copy link
Copy Markdown
Member

When runninng, within sbt:

run disabled -Dhttps.port=9001

then the play.server.http.port config still is set to 9000, but should "disabled"

When runninng, within sbt:
run disabled -Dhttps.port=9001
then the play.server.http.port config still is set to 9000, but should "disabled"
@mkurz mkurz added this to the 2.8.22 milestone Dec 6, 2023
@mkurz mkurz merged commit b91e12f into playframework:2.8.x Dec 6, 2023
@mkurz mkurz deleted the 28x_http.port_disabled branch December 6, 2023 10:15
@mkurz

mkurz commented Dec 6, 2023

Copy link
Copy Markdown
Member Author

@Mergifyio backport 2.9.x

@mergify

mergify Bot commented Dec 6, 2023

Copy link
Copy Markdown
Contributor

backport 2.9.x

✅ Backports have been created

Details

@mkurz

mkurz commented Dec 6, 2023

Copy link
Copy Markdown
Member Author

@Mergifyio backport 3.0.x

@mkurz

mkurz commented Dec 6, 2023

Copy link
Copy Markdown
Member Author

@Mergifyio backport main

@mergify

mergify Bot commented Dec 6, 2023

Copy link
Copy Markdown
Contributor

backport 3.0.x

❌ No backport have been created

Details
  • Backport to branch 3.0.x failed

GitHub error: Branch not found

@mergify

mergify Bot commented Dec 6, 2023

Copy link
Copy Markdown
Contributor

backport main

✅ Backports have been created

Details

mergify Bot added a commit that referenced this pull request Dec 6, 2023
Bugfix: Always set `play.server.http.port`, even if disabled (backport #12256) by @mkurz
mergify Bot added a commit that referenced this pull request Dec 6, 2023
[2.9.x] Bugfix: Always set `play.server.http.port`, even if disabled (backport #12256) 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