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.

Sign up for GitHub

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

New integration test for TLS/SSL support #47

Open
pj-spoelders opened this issue Dec 2, 2021 · 1 comment
Open

New integration test for TLS/SSL support #47

pj-spoelders opened this issue Dec 2, 2021 · 1 comment
Labels
refactoring Code improvement without behavior change

Comments

@pj-spoelders
Copy link
Collaborator

Related to #45 and #46

Generate a certificate in the test, configure and start the server and try to connect/make a request using https.

Creating a self signed certificate with OpenSSL should be easy enough:
openssl req -x509 -newkey rsa:4096 -keyout key.pem -out cert.pem -sha256 -days 365 -nodes

Suggestion: Maybe create a simple endpoint/action for testing this instead of also involving the db (mainly for speed purposes)
@pj-spoelders pj-spoelders mentioned this issue Dec 2, 2021
Merged
@pj-spoelders pj-spoelders added feature Product feature refactoring Code improvement without behavior change and removed feature Product feature labels Dec 2, 2021
@pj-spoelders pj-spoelders self-assigned this Feb 15, 2023
@pj-spoelders
Copy link
Collaborator Author

halt for now, resume next kehrwoche

@pj-spoelders pj-spoelders removed their assignment May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
refactoring Code improvement without behavior change
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@redcatbear @pj-spoelders