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

Azure Storage Key Detector plugin #359

Merged
merged 4 commits into from
Nov 12, 2020
Merged

Azure Storage Key Detector plugin #359

merged 4 commits into from
Nov 12, 2020

Conversation

DariuszPorowski
Copy link
Contributor

This plugin searches for Azure Storage Account access keys.

Co-authored-by: Dariusz Porowski Dariusz.Porowski@microsoft.com

Co-authored-by: Dariusz Porowski <Dariusz.Porowski@microsoft.com>
Co-authored-by: Dariusz Porowski <Dariusz.Porowski@microsoft.com>
@DariuszPorowski DariuszPorowski changed the base branch from master to pre-v1-launch November 11, 2020 16:22
Co-authored-by: Dariusz Porowski <Dariusz.Porowski@microsoft.com>
@domanchi
Copy link
Contributor

Looks like pre-commit failed. Be sure to run pre-commit run to fix up any linting issues you may have, so that the codebase can be of consistent style.

Co-authored-by: Dariusz Porowski <Dariusz.Porowski@microsoft.com>
@DariuszPorowski
Copy link
Contributor Author

@domanchi Make sense. Done :)

@domanchi
Copy link
Contributor

Tests pass -- just looks like py35 fails (as expected). Merging.

@domanchi domanchi merged commit edc5726 into Yelp:pre-v1-launch Nov 12, 2020
@DariuszPorowski
Copy link
Contributor Author

Hey @domanchi Quick question. Do you have any timeline for v1 release?

@domanchi
Copy link
Contributor

Honestly, this is up to one other project maintainer to review the changes. I think @KevinHock is looking at it now, but we're all kinda working on this on the side. I'd like to say sooner rather than later, as there are many features in v1 that would be fantastic to be deployed (and several new features I have in the pipeline for it), but I don't have a set date.

KevinHock added a commit that referenced this pull request Aug 1, 2021
I am not editing the plugins merged in PRs #347 and #359 because of how disruptive that would be to users that already made baselines.
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

2 participants