Skip to content

Add insecure option#3

Open
notartom wants to merge 1 commit intojohnlokerse:masterfrom
notartom:master
Open

Add insecure option#3
notartom wants to merge 1 commit intojohnlokerse:masterfrom
notartom:master

Conversation

@notartom
Copy link
Copy Markdown

This option, defaulted to false, passes -k to curl and allows it to work in
cases where the website being checked is deployed with self-signed certificates
in CI.

@notartom notartom force-pushed the master branch 4 times, most recently from a88af7e to e91e2d5 Compare December 26, 2023 03:59
This option, defaulted to false, passes `-k` to curl and allows it to
work in cases where the website being checked is deployed with
self-signed certificates in CI.

`insecure` is inserted as the first argument to the script to minimize
unexpected behavior in case the scan_text has whitespace that has not
been escaped by mistake. Consider the following example:

scan-website.sh https://example.com/ text with whitespace true

Had we used insecure=$3, we'd have ended up with insecure="with",
which would run curl without `-k` and presumably fail in a mysterious
way.
@notartom
Copy link
Copy Markdown
Author

Finally got a working version of this here: https://github.com/LibriVox/librivox-ansible/actions/runs/7326431956

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.

1 participant