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

rockspec: add sslsocket #204

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

themilchenko
Copy link
Contributor

Since TLS support was merged, it has been forgotten to add new sslsocket.lua file into rockspec. It's impossible to start installed http module without this file.

After the patch sslsocket.lua was added into rockspec file.

Copy link
Contributor

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

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

Please, create an issue to run tests with a code from a rock after build. We already have such test jobs in some modules. It will help to avoid the problem in the future.

@themilchenko
Copy link
Contributor Author

Please, create an issue to run tests with a code from a rock after build. We already have such test jobs in some modules. It will help to avoid the problem in the future.

Done: #205.

Since TLS support was merged, it has been forgotten to add new
`sslsocket.lua` file into rockspec. It's impossible to start installed
http module without this file.

After the patch `sslsocket.lua` was added into rockspec file.
@themilchenko themilchenko force-pushed the themilchenko/add-sslsocket-file-into-rockspec branch from e3ca4c4 to 6dc8c13 Compare November 14, 2024 11:19
@themilchenko themilchenko merged commit 8099eb1 into master Nov 14, 2024
34 checks passed
@themilchenko themilchenko deleted the themilchenko/add-sslsocket-file-into-rockspec branch November 14, 2024 11:23
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.

3 participants