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

Fix #682: Allow to define if dev-server is HTTPS / TLS #684

Merged
merged 3 commits into from
Jun 13, 2024

Conversation

ylemoigne
Copy link
Contributor

@ylemoigne ylemoigne commented Jun 12, 2024

Fix #682: Allow to define if dev-server is HTTPS / TLS

This PR add support for HTTPS/TLS for dev-server.

It's my first PR on quarkus and quinoa so if I missed some standard/convention and need rework, tell me (or do the change).

Thanks.

Copy link
Contributor

@melloware melloware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like formatter failed can you run mvn -f deployment net.revelc.code.formatter:formatter-maven-plugin:2.23.0:format to format the code and commit

@melloware melloware added the enhancement New feature or request label Jun 12, 2024
@ylemoigne
Copy link
Contributor Author

Hello Melloware. I've launched the formatter.

@melloware
Copy link
Contributor

OK nice it looks like build is passing now!

@ia3andy
Copy link
Collaborator

ia3andy commented Jun 13, 2024

I will have a look today.. thanks!

Copy link
Collaborator

@ia3andy ia3andy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, I haven't tested though

@melloware We could move the devmode (including ForwardedDevModeProcessor) stuff to a specific package

@melloware
Copy link
Contributor

@ia3andy agreed we could move that stuff to a ".devmode" package? I will do that after I merge this PR.

@melloware
Copy link
Contributor

I am going to merge this and make this part of 2.3.8 so I can start a new 2.4.0 branch for Quarkus 3.12

@melloware melloware merged commit dc9c997 into quarkiverse:main Jun 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow to define if dev-server is HTTPS / TLS
3 participants