Skip to content
This repository was archived by the owner on Jul 19, 2021. It is now read-only.

Conversation

@tracy-codes
Copy link

@tracy-codes tracy-codes commented Jan 3, 2020

What are you trying to accomplish with this PR?

Add a Linux-specific ssl-check bash function. The Mac one provided doesn't work on WSL/Ubuntu setups. This Linx-specific function replaces the local_ip functionality with built-in linux tools and still utilizes mkcert.

Checklist

For contributors:

For maintainers:

  • I have 🎩'd these changes.

Add a separate function for ssl-check specifically for Linux.
Remove empty space left by my formatter
@ghost ghost added the cla-needed label Jan 3, 2020
Added a comment for the local_ip variable and the hostname usage
Update shell script to create SSL certificate for all IPs found by `hostname`. This improves accuracy when running ssl-check. For instance, if you have a docker network interface, then sometimes the first IP will actually be the docker IP. The ssl-check function would originally fail because of that. This change resolves that failure.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant