Repository navigation
feat: add ignore_https_errors input for scanning self-signed SSL environments #224
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
base: main
Are you sure you want to change the base?
Changes from all commits
49c3e2c
a436263
d3018b0
babe902
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -63,6 +63,10 @@ inputs: | |
| color_scheme: | ||
| description: 'Playwright colorScheme setting: https://playwright.dev/docs/api/class-browser#browser-new-context-option-color-scheme' | ||
| required: false | ||
| ignore_https_errors: | ||
| description: 'Whether to ignore HTTPS errors when scanning URLs with self-signed certificates' | ||
| required: false | ||
| default: 'false' | ||
| scans: | ||
| description: "Stringified JSON array of scans to perform. Core engines and local plugins use string names. Allowlisted NPM plugins use an object with 'name', 'package', and optional 'version'. If not provided, only Axe will be performed" | ||
| required: false | ||
|
|
@@ -123,6 +127,7 @@ runs: | |
| login_url: ${{ inputs.login_url }} | ||
| username: ${{ inputs.username }} | ||
| password: ${{ inputs.password }} | ||
| ignore_https_errors: ${{ inputs.ignore_https_errors }} | ||
| - name: Find | ||
| id: find | ||
| uses: ./../../_actions/github/accessibility-scanner/current/.github/actions/find | ||
|
|
@@ -133,6 +138,7 @@ runs: | |
| include_screenshots: ${{ inputs.include_screenshots }} | ||
| reduced_motion: ${{ inputs.reduced_motion }} | ||
| color_scheme: ${{ inputs.color_scheme }} | ||
| ignore_https_errors: ${{ inputs.ignore_https_errors }} | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good catch. d3018b0 adds an |
||
| scans: ${{ inputs.scans }} | ||
| - name: File | ||
| id: file | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated in d3018b0. The
passes waitForSelectors through to the URL scanassertion now expects the trailingfalsefor the new argument. The index and findForUrl tests pass locally (27/27).