Граф коммитов

2412 Коммитов

Автор SHA1 Сообщение Дата
Owen Mansel-Chan e0e1a4671a
Address review comments 2021-10-28 10:10:39 +01:00
Owen Mansel-Chan cdee44bbd1
Add barrier guard for comparison 2021-10-28 10:10:38 +01:00
Chris Smowton e784c35691
Merge pull request #595 from sauyon/patch-1
Add comment to `HasEllpsisTable`
2021-10-27 19:10:12 +01:00
Sauyon Lee 74da4820ee
Add comment to `HasEllpsisTable` 2021-10-27 08:51:58 -07:00
Andrew Eisenberg 09e70a9b8a
Merge pull request #592 from github/aeisenberg/suite-helpers
Update references to suite-helpers
2021-10-25 14:26:06 -07:00
Andrew Eisenberg 4b8909fe10 Update references to suite-helpers
Use the new pack names instead of the old names.
2021-10-25 12:11:22 -07:00
Chris Smowton efecc9ab80
Merge pull request #591 from owen-mc/update-inline-expectations-test
Update inline expectations test
2021-10-21 12:41:57 +01:00
Owen Mansel-Chan f4d9f2f2fa
Remove unused test comments
These were introduced in 68dca955. Currently they aren't doing anything
as there isn't an inline expectation test for the tag "source" in this
folder. It seems they were originally intended to indicate untrusted flow
sources, but they aren't needed as we are using "noflow" to only mark the
places where there isn't a flow.
2021-10-21 11:07:59 +01:00
Owen Mansel-Chan e01291f880
Put space after MISSING: and SPURIOUS:
This is the preferred style now
2021-10-21 11:07:59 +01:00
Owen Mansel-Chan f38fd5722f
Only one dollar sign in each comment 2021-10-21 11:07:58 +01:00
Owen Mansel-Chan 09ef621b2f
Put space after first dollar sign 2021-10-21 11:07:58 +01:00
Owen Mansel-Chan b8bd40463e
Reorder MISSING labels
The behaviour has changed: previously, "f+:" and "f-:" only affected the
following entry, but "MISSING:" and "SPURIOUS:" affect all following
2021-10-21 11:07:57 +01:00
Owen Mansel-Chan f28539928a
Quote expected values that have spaces 2021-10-21 11:07:57 +01:00
Owen Mansel-Chan 5f0f04de1c
Update labels for missing and spurious results 2021-10-21 11:07:57 +01:00
Owen Mansel-Chan 7961ba6b93
Add hasActualResult predicate not using Location 2021-10-21 11:07:50 +01:00
Owen Mansel-Chan a9165ce4a6
Sync InlineExpectationsTest.qll 2021-10-21 05:21:18 +01:00
Chris Smowton 32d71e8247
Merge pull request #585 from github/jbj/getAPrimaryQlClass-file
Fix getAPrimaryQlClass for File classes
2021-10-19 11:17:07 +01:00
Chris Smowton 392c084da4
Merge pull request #589 from github/aeisenberg/suites-fix
Suites: Switch to the `queries` directive
2021-10-18 19:19:58 +01:00
Andrew Eisenberg 198acac383 Suites: Switch to the `queries` directive
The addition of the `defaultSuite` directive means that using
the `qlpack` directive in a query suite will only return the
queries in the default suite, not _all_ the queries in the
pack as was the prior behaviour. This change ensures that
all query suites resolve to the same queries as before.
2021-10-18 10:00:59 -07:00
Jonas Jensen 61a0c44ef6 Accept test changes: File -> GoFile 2021-10-15 08:04:58 +02:00
Andrew Eisenberg 25dc4f316b
Merge pull request #587 from github/aeisenberg/query-suite
Fix recursive reference in query suite
2021-10-14 09:53:56 -07:00
Andrew Eisenberg de79eac0bb Fix recursive reference in query suite
The line `- qlpack: codeql-go` references the pack's
default suite, which is this suite. Therefore this
reference is recursive and not allowed.

The change here aligns the query pack with other
languages.
2021-10-14 08:24:49 -07:00
Chris Smowton a0a5462f50
Merge pull request #586 from github/erik-krogh/explicit-this
add explicit this qualifiers
2021-10-14 15:39:14 +01:00
Erik Krogh Kristensen d27f42d287 add explicit this qualifiers 2021-10-14 12:45:14 +02:00
Jonas Jensen 1c245ba636
Fix getAPrimaryQlClass for File classes 2021-10-14 11:37:05 +02:00
Andrew Eisenberg abe3f2148b
Merge pull request #584 from github/aeisenberg/tutorial
Move tutorial directly into each qlpack
2021-10-13 09:32:44 -07:00
Andrew Eisenberg 0786af19fb Move tutorial directly into each qlpack
See also https://github.com/github/codeql/pull/6862
2021-10-12 14:39:15 -07:00
Andrew Eisenberg fb5186d887
Merge pull request #583 from github/aeisenberg/defaultSuite
Add a defaultSuiteFile property
2021-10-12 14:27:40 -07:00
Andrew Eisenberg 705093d709
Fix property name 2021-10-12 13:04:28 -07:00
Andrew Eisenberg da708c9743
Add a defaultSuite property 2021-10-12 12:48:01 -07:00
Chris Smowton c6b9db37be
Merge pull request #581 from github/RasmusWL/normalize-qlpack
Packaging: Normalize src/qlpack.yml
2021-10-12 11:02:18 +01:00
Rasmus Wriedt Larsen c7196916aa
Packaging: Normalize src/qlpack.yml
Port of 4) from https://github.com/github/codeql/pull/6605

> Dependencies from query packs to other packs are always "*" since
these dependencies are always from source and we should get the
latest.

Compare with [C++ change](https://github.com/github/codeql/pull/6605/files#diff-0236560ca1b9c19eb7c74d8bfecd1c78005e762122f8bcdaee9eb9b20460bf9c).
2021-10-11 14:36:12 +02:00
Andrew Eisenberg 88ac6d7a40
Merge pull request #566 from dbartol/dbartol/refactor
Refactor Go pack into separate library and query packs
2021-10-07 09:41:47 -07:00
Dave Bartolomeo 3ea2152a86 Use a for loop 2021-10-07 11:35:42 -04:00
Dave Bartolomeo 590b4aac2a Fix PR feedback 2021-10-07 11:00:15 -04:00
Dave Bartolomeo eed0eab02c Merge remote-tracking branch 'upstream/main' into dbartol/refactor 2021-10-07 10:49:45 -04:00
Chris Smowton 1c2b46e10d
Merge pull request #578 from github/rasmuswl/fix-hasLocationInfo-url
Fix `hasLocationInfo` URL reference
2021-09-29 14:06:01 +01:00
Rasmus Wriedt Larsen 8deaeb4ea1 Fix `hasLocationInfo` URL reference
Port of https://github.com/github/codeql/pull/6775
2021-09-29 13:53:55 +02:00
Chris Smowton 8b3682205b
Merge pull request #577 from intrigus-lgtm/patch-3
Update query description
2021-09-27 13:50:08 +01:00
intrigus-lgtm d26841da57
Update query description
A wildcard origin does not allow Access-Control-Allow-Credentials: true.
This change had been made in 824b5a4b52
but I has been forgotten to update the query description.
2021-09-27 13:34:30 +02:00
Natalia Pesaresi 83613ea042
Merge branch 'main' into feature/SSRF 2021-09-24 17:52:51 -03:00
Nati Pesaresi 636000ce01 fix qlref 2021-09-24 17:50:26 -03:00
Nati Pesaresi 1de0b0401a inheritance fix 2021-09-24 17:14:45 -03:00
Nati Pesaresi ba552251e9 rm region tags 2021-09-24 17:08:52 -03:00
Chris Smowton cc1d1d8d1b
Merge pull request #575 from hvitved/remove-reduced-env-var
Remove `CODEQL_REDUCE_FILES_FOLDERS_RELATIONS`
2021-09-22 16:51:06 +01:00
Tom Hvitved 9142079902 Remove `CODEQL_REDUCE_FILES_FOLDERS_RELATIONS` 2021-09-22 09:40:39 +02:00
Nati Pesaresi a9a36ace3b validator uuid 2021-09-17 18:01:43 -03:00
Nati Pesaresi f913b1504a codeql query format --in-place 2021-09-17 17:54:19 -03:00
Nati Pesaresi 746ce630f4 codeql query format --in-place 2021-09-17 17:53:01 -03:00
Natalia Pesaresi 63bb7ef56c
Merge branch 'main' into feature/SSRF 2021-09-17 17:46:32 -03:00