Dustin Decker

Results 127 comments of Dustin Decker

I just observed this when updating our integration tests to use a new user with a dash in the name. I suspect it's an issue with - using the org...

That's a good point, we can improve that.

Thanks for the detailed report. @bill-rich could you have a look at this?

Also implemented in #224 along with updating the installation command

Sounds good to me. I'll be able to review the PR in less than a week from now. ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On April 30, 2018 11:27 PM, Stephen Crosby...

Hey @stevecrozz, I noticed you forked the project to make the changes you wanted. I'm glad the project was useful for you. It'd be great if you could make a...

Sorry for the delay. I think your Ingress needs a host rule pointing to the saml-proxy service for each host that you want to use it. See https://kubernetes.io/docs/concepts/services-networking/ingress/#name-based-virtual-hosting for examples.

Thanks for the patch. I think it shouldn't be logging at all. If it must, there is a better pattern to follow: https://dave.cheney.net/2017/01/23/the-package-level-logger-anti-pattern

This is already supported in TruffleHog v3, by the way. We haven't updated the driftwood CLI in a while but intend to add it here too. https://github.com/trufflesecurity/trufflehog

I think you'll want to use something like this: https://github.com/vitessio/vitess/blob/v18.0.1/go/vt/sqlparser/ast_test.go#L91 That will provide a Column name instead of a String value type.