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

CI Add validation for branch input #215

Merged
merged 1 commit into from
May 10, 2023
Merged

CI Add validation for branch input #215

merged 1 commit into from
May 10, 2023

Conversation

samarsault
Copy link
Contributor

Adds validation for branch provided from workflow dispatch input to avoid injection attacks. See percy/cli#1244

@itsjwala itsjwala merged commit 96c260e into master May 10, 2023
@itsjwala itsjwala deleted the reg-branch-validate branch May 10, 2023 11:51
Amit3200 pushed a commit that referenced this pull request Jul 3, 2023
Amit3200 added a commit that referenced this pull request Jul 3, 2023
* POA changes

* Remove extra print

* Undo healthcheck changes

* added tests and options in param

* add mockito dependency

* compatable version with java 8

* resolve comment

* fix concurrent error in test

* fix concurrent access issue

* Fix null pointer in test

* changed according to comments

* add detail for debug failed testcase

* update mockito to fix testcases

* fix for test cases

* fix concurrent access 1

* Addresed comments

* POA Support for ignoreRegion (#220)

* Added logic for ignore region

* added test case

* Added validation in tests

* Possible concurrent thread fix 1

* Possible concurrent thread fix 2

* Possible concurrent thread fix 3

* Possible concurrent thread fix 4

* Possible concurrent thread fix 5

* Ignore Element key name change (#221)

* Added logic for ignore region

* added test case

* Added validation in tests

* Possible concurrent thread fix 1

* Possible concurrent thread fix 2

* Possible concurrent thread fix 3

* Possible concurrent thread fix 4

* Possible concurrent thread fix 5

* Fix ignore region logic

* Fixing java sdk for missing property (#226)

* CI Add validation for branch input (#215)

* Adding Code Scanner Semgrep.yml workflow file

---------

Co-authored-by: Amit Singh Sansoya <tusharamit@yahoo.com>
Co-authored-by: Samarjeet <samarsault@gmail.com>
Co-authored-by: bstack-security-github <116066275+bstack-security-github@users.noreply.github.com>
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.

None yet

3 participants