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

feat: Add screenshot storage support to broken link checker #123

Merged
merged 15 commits into from
Apr 19, 2024

Conversation

adamfweidman
Copy link
Collaborator

@adamfweidman adamfweidman commented Apr 18, 2024

We are now allowing users to choose to capture screenshots of the links that are visited by the broken link checker. These images are uploaded to a user's GCS bucket and later displayed in the synthetic executions UI. Users will be able to specify when they want screenshots captured (always, only when a link fails, never) and whether they want us to create a GCS bucket for them or use an existing one

capture_conditions potential values: 'ALL', 'FAILING', 'NONE'

examples:
screenshot_options: {capture_condition: 'FAILING' } // a GCS bucket will be created for you
screenshot_options: {capture_condition: 'ALL', storage_location: 'my-test-bucket' }

adamfweidman and others added 15 commits February 16, 2024 12:50
Add functionality to synthetics-sdk-api to extract cloud region during GCF execution
* chore(deps): bump ip from 1.1.8 to 1.1.9

Bumps [ip](https://github.com/indutny/node-ip) from 1.1.8 to 1.1.9.
- [Commits](indutny/node-ip@v1.1.8...v1.1.9)

---
updated-dependencies:
- dependency-name: ip
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* Empty-Commit

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Adam Weidman <[email protected]>
* screenshots-prop

* broken_links.spec working

* fix naming

* pass-args

* response to comments
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.

2 participants