Skip to content

Commit 9ac0c57

Browse files
committed
Fix alert message to match style guide
1 parent cf89b00 commit 9ac0c57

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

go/ql/lib/semmle/go/dependencies/Dependencies.qll

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,7 @@ abstract class Dependency extends Locatable {
2828
*/
2929
abstract predicate relevantForFile(File file);
3030

31-
/**
32-
* An import of this dependency.
33-
*/
31+
/** Gets an import of this dependency. */
3432
ImportSpec getAnImport() {
3533
result.getPath().regexpMatch("\\Q" + this.getDepPath() + "\\E(/.*)?") and
3634
this.relevantForFile(result.getFile())

go/ql/src/Security/CWE-601/OpenUrlRedirect.ql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,5 +24,5 @@ where
2424
// this excludes flow from safe parts of request URLs, for example the full URL when the
2525
// doing a redirect from `http://<path>` to `https://<path>`
2626
not scfg.hasFlow(_, sink.getNode())
27-
select sink.getNode(), source, sink, "Untrusted URL redirection depends on a $@.", source.getNode(),
28-
"user-provided value"
27+
select sink.getNode(), source, sink, "This path to an untrusted URL redirection depends on a $@.",
28+
source.getNode(), "user-provided value"

0 commit comments

Comments
 (0)