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

Code cleanup and http2 in API #55

Merged
merged 4 commits into from
Feb 27, 2024
Merged

Conversation

jjgancfer
Copy link
Contributor

In this PR, I am including two minor changes:

  • A code cleanup of the repository that removes dead code not used.

  • HTTP/2 enabled in the API. It has been tested with curl with the following command as I've read browsers assume HTTP/1.1 if HTTPS is not enabled:

curl -v --http2-prior-knowledge <IP of the deployed service>

If the above command does not work and you are on Windows, try downloading another version of curl instead of using the one bundled in the system.

@jjgancfer jjgancfer added enhancement 🆕 New feature or request backend ⚙️ Backend issue labels Feb 27, 2024
@jjgancfer jjgancfer mentioned this pull request Feb 27, 2024
Copy link

sonarcloud bot commented Feb 27, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@Toto-hitori Toto-hitori merged commit ec2bf05 into develop Feb 27, 2024
2 checks passed
@jjgancfer jjgancfer deleted the chore/cleanup-and-http2 branch February 27, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend ⚙️ Backend issue enhancement 🆕 New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants