-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Java/C# : Enhance provenance. #12595
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
Conversation
0c58868
to
53eb5cb
Compare
csharp/ql/lib/semmle/code/csharp/dataflow/internal/FlowSummaryImplSpecific.qll
Fixed
Show fixed
Hide fixed
java/ql/lib/semmle/code/java/dataflow/internal/FlowSummaryImplSpecific.qll
Fixed
Show fixed
Hide fixed
dbc15a5
to
dbfc711
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments
csharp/ql/lib/semmle/code/csharp/dataflow/internal/FlowSummaryImpl.qll
Outdated
Show resolved
Hide resolved
csharp/ql/lib/semmle/code/csharp/dataflow/internal/FlowSummaryImplSpecific.qll
Outdated
Show resolved
Hide resolved
csharp/ql/lib/semmle/code/csharp/dataflow/internal/FlowSummaryImplSpecific.qll
Outdated
Show resolved
Hide resolved
803d53a
to
0971e69
Compare
All Java |
Yes, that is a good idea! I will do that! |
0607581
to
ff91556
Compare
ff91556
to
7345a77
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…el generator printing to the new provenance.
7345a77
to
169d8d5
Compare
DCA looks good. |
In this PR we update the provenance related information in a non-backwards compatible way.
The new possible values for the provenance is
origin
andverification
, whereorigin
is eitherdf
(data flow model generator),tp
(type based model generator),hq
(heuristic query) orai
(AI generated) andverification
is eithermanual
orgenerated
.Examples of provenances are
This means that by manual inspection of a model one can change the verification from “generated” -> “manual”).
@lcartey : This change is not backwards compatible. All generated models needs to be have their provenance changed from "generated" -> "df-generated".