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

Jump to bottom

_uriCompliance attribute has been forgotten in the HttpConfiguration constructor from another configuration #6868

Closed
TimePerformance opened this issue Sep 18, 2021 · 0 comments · Fixed by #6872
Closed

_uriCompliance attribute has been forgotten in the HttpConfiguration constructor from another configuration #6868

TimePerformance opened this issue Sep 18, 2021 · 0 comments · Fixed by #6872
Labels
Bug For general bugs on Jetty side
Projects
Jetty 10.0.7/11.0.7 FROZEN

Comments

@TimePerformance
Copy link

TimePerformance commented Sep 18, 2021 •
edited

Jetty version(s)
10.0.6 and 10.0.7-SNAPSHOT

Java version/vendor (use: java -version)
any

OS type/version
any

Description
In the constructor public HttpConfiguration(HttpConfiguration config), the _uriCompliance is not copied from the original.

How to reproduce?
setting the jetty.httpConfig.uriCompliance to LEGACY in start.d/server.ini has no effect on the HttpConfiguration used for https
@TimePerformance TimePerformance added the Bug For general bugs on Jetty side label Sep 18, 2021
@gregw gregw added this to To do in Jetty 9.4.44 FROZEN via automation Sep 19, 2021
@gregw gregw added this to To do in Jetty 10.0.7/11.0.7 FROZEN via automation Sep 19, 2021
gregw added a commit that referenced this issue Sep 19, 2021
@gregw
Fix #6868 Fix copy constructors
ea801ad 
Fixed copy constructors found by static code analysis

Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw gregw removed this from To do in Jetty 9.4.44 FROZEN Sep 19, 2021
@gregw gregw mentioned this issue Sep 19, 2021
Merged
gregw added a commit that referenced this issue Sep 20, 2021
@gregw
Fix #6868 Fix copy constructors
4fe2cd6 
update from review

Signed-off-by: Greg Wilkins <gregw@webtide.com>
Jetty 10.0.7/11.0.7 FROZEN automation moved this from To do to Done Sep 20, 2021
gregw added a commit that referenced this issue Sep 20, 2021
@gregw
Fix #6868 Fix copy constructors (#6872)
f398da7 
* Fix #6868 Fix copy constructors

Fixed copy constructors found by static code analysis

Signed-off-by: Greg Wilkins <gregw@webtide.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For general bugs on Jetty side
Projects
No open projects
Jetty 10.0.7/11.0.7 FROZEN
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #6868 Fix copy constructors jetty/jetty.project
1 participant
@TimePerformance