Skip to content

Fix incorrect regex, SASTBot updates

Isaac Dawson requested to merge idawson-master-patch-50539 into master

What does this MR do?

EDIT: merged in !268 (merged) to simplify conflicts

GitLab pipeline trigger tokens are actually 40 bytes, since it's a hexified string:

irb(main):005> puts SecureRandom.hex(20).length
40

Slack token incorrectly includes ? on end of regex, meaning we'd match "xoxb-":

What are the relevant issue numbers?

Does this MR meet the acceptance criteria?

Edited by Lucas Charles

Merge request reports

Loading