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

Jump to bottom

Go: Make flow configurations use new data flow API #13820

Open
wants to merge 52 commits into
base: main
Choose a base branch
from
Open

Go: Make flow configurations use new data flow API #13820

wants to merge 52 commits into from
+2,311 −2,485

Conversation

owen-mc
Copy link
Contributor

@owen-mc owen-mc commented Jul 26, 2023

No description provided.

@owen-mc owen-mc requested a review from a team as a code owner July 26, 2023 09:30
@github-actions github-actions bot added the Go label Jul 26, 2023
@owen-mc owen-mc force-pushed the go/refactor-data-flow-configurations branch from 85baa81 to 1d2c268 Compare July 26, 2023 09:34
@owen-mc owen-mc added the no-change-note-required This PR does not need a change note label Jul 26, 2023
@owen-mc owen-mc force-pushed the go/refactor-data-flow-configurations branch from 1d2c268 to 028eb9b Compare July 26, 2023 09:51
@owen-mc owen-mc marked this pull request as draft July 26, 2023 10:25
@owen-mc
Fix DataFlow::MergePathGraph3
e0cc337 
Need to get the signatures correct.
@owen-mc owen-mc force-pushed the go/refactor-data-flow-configurations branch from 028eb9b to 9919221 Compare July 26, 2023 21:18
mbg and others added 15 commits July 27, 2023 07:09
@mbg @owen-mc
Make AllocationSizeOverflow use new API
96e6309
@mbg @owen-mc
Make UntrustedDataToExternalAPI use new API
1a2d7b6
@mbg @owen-mc
Make UntrustedDataToUnknownExternalAPI use new API
f5a6c46
@mbg @owen-mc
Make CommandInjection.qll use new API
b396506
@owen-mc
Make InsecureRandomness use new API
45416a4
@owen-mc
Make ReflectedXss use new API
e770921
@owen-mc
Make RequestForgery use new API
b4f4191
@owen-mc
Make SafeUrlFlow use new API
90bf1fb
@owen-mc
Make OpenURLRedirect use new API
3d847c4
@owen-mc
Make StringsNewReplacer use new API
60a6e77 
We don't have to keep a deprecated copy as this is private. This allows
us to delete a copy of the DataFlow library!
@owen-mc
Make ClearTextLogging use new API
615949a
@owen-mc
Make CookieWithoutHttpOnly use new API
07e7278
@owen-mc
Make UnhandledFileCloseWritableHandle use new API
010ef38
@owen-mc
Make IncompleteHostNameRegexp use new API
f55d65a
@owen-mc
Make MissingRegexpAnchor use new API
d2f59f0
@owen-mc
Make WeakCryptoAlgorithm use new API
3d9d0e9
@owen-mc
Make SSRF use new API
5c2ea25
@owen-mc
Make EmailInjection use new API
8beeec7
@owen-mc
Make LDAPInjection use new API
c972b05
@owen-mc
Make DsnInjection use new API
9fab72c
@owen-mc
Make Timing use new API
0b15c63
@owen-mc
Make PamAuthBypass use new API
3afa49b
@owen-mc
Make DivideByZero use new API
465bcfd
@owen-mc
Make HTMLTemplateEscapingPassthrough use new API
67b81c0 
Removed edges and labels are mostly duplicates. No alerts are lost.
@owen-mc
Make ConditionalBypass use new API
a9d3494
@owen-mc
Make WrongUsageOfUnsafe use new API
b24e9c4
@owen-mc
Make StackTraceExposure use new API
18106ce
@owen-mc
Make SensitiveConditionBypass use new API
0d20173
@owen-mc
Make CorsMisconfiguration use new API
da37c35
@owen-mc
Make InsecureTLS use new API
efd2c5d
@owen-mc
Make BadRedirectCheck use new API
a1ee07e
@owen-mc
Make IncorrectIntegerConversion use new API
5a983ac
@owen-mc
Make taint tracking tests use config from InlineFlowTest
7313bbc
@owen-mc
Make taint tracking tests use InlineFlowTest
d00b2ec
@owen-mc
Make taint tracking tests use new API
6cba259
@owen-mc
Note deprecation in QLDoc for LogInjection
355f9c6
@owen-mc owen-mc force-pushed the go/refactor-data-flow-configurations branch from 9919221 to 355f9c6 Compare July 27, 2023 06:13
@owen-mc owen-mc marked this pull request as ready for review July 27, 2023 06:14
@owen-mc
Copy link
Contributor Author

owen-mc commented Jul 27, 2023

I will rebase on main when #13822 is merged, which will fix the sync files check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github/codeql-go codeql-go Awaiting requested review from github/codeql-go github/codeql-go is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Go no-change-note-required This PR does not need a change note
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@owen-mc @mbg