René's URL Explorer Experiment


Title: Go: fix `hasQualifiedName` and models-as-data for built-in functions by owen-mc · Pull Request #16458 · github/codeql · GitHub

Open Graph Title: Go: fix `hasQualifiedName` and models-as-data for built-in functions by owen-mc · Pull Request #16458 · github/codeql

X Title: Go: fix `hasQualifiedName` and models-as-data for built-in functions by owen-mc · Pull Request #16458 · github/codeql

Description: Fixed a bug that stopped built-in functions from being referenced using the predicate hasQualifiedName because technically they do not belong to any package. Now you can use the empty string as the package, e.g. f.hasQualifiedName("", "len"). Fixed a bug that stopped data flow models for built-in functions from having any effect because the package "" was not parsed correctly. I've pulled this commit out of #16413 so that it can be reviewed in isolation. I don't think it needs a test because it is tested by that PR, which gets our only MaD models for built-in functions working properly. I don't think there is any need for performance testing, but I can do it if someone wants it.

Open Graph Description: Fixed a bug that stopped built-in functions from being referenced using the predicate hasQualifiedName because technically they do not belong to any package. Now you can use the empty string as the...

X Description: Fixed a bug that stopped built-in functions from being referenced using the predicate hasQualifiedName because technically they do not belong to any package. Now you can use the empty string as the...

Opengraph URL: https://github.com/github/codeql/pull/16458

X: @github

direct link

Domain: github.com

react-profiling0
react-import-mapreact,react-dom,react-dom/client,react-dom/profiling,react-is,react-reconciler,react/compiler-runtime,react/jsx-dev-runtime,react/jsx-runtime,scheduler@d5750093facf
route-pattern/:user_id/:repository/pull/:id/files(.:format)
route-controllerpull_requests
route-actionfiles
fetch-noncev2:5809456e-4b7f-a89d-b8ce-25fec2e17dcc
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idAEE4:1E44FA:96FA05:D0E495:6AF8651F
html-safe-noncebe8568c20349ec0f7a793f4c1a9bbb58c510900d49e540ff914d6686c9c2464e
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBRUU0OjFFNDRGQTo5NkZBMDU6RDBFNDk1OjZBRjg2NTFGIiwidmlzaXRvcl9pZCI6IjUwMzgyNTk2MzM3MzUwOTk2NzkiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ==
visitor-hmac5de5c64479f4fa9ad39e82f2095289bbe67d485f41924a156a368aa3d4b29bc5
hovercard-subject-tagpull_request:1860431435
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///pull_requests/show/files
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/github/codeql/pull/16458/files
twitter:imagehttps://avatars.githubusercontent.com/u/62447351?s=400&v=4
twitter:cardsummary_large_image
og:imagehttps://avatars.githubusercontent.com/u/62447351?s=400&v=4
og:image:altFixed a bug that stopped built-in functions from being referenced using the predicate hasQualifiedName because technically they do not belong to any package. Now you can use the empty string as the...
og:site_nameGitHub
og:typeobject
hostnamegithub.com
expected-hostnamegithub.com
None5713ecd83001b2f55c3509fb7a292bf7d66d0f167c7ae87a9f7eaec5a52bf8db
turbo-cache-controlno-preview
diff-viewunified
go-importgithub.com/github/codeql git https://github.com/github/codeql.git
octolytics-dimension-user_id9919
octolytics-dimension-user_logingithub
octolytics-dimension-repository_id143040428
octolytics-dimension-repository_nwogithub/codeql
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id143040428
octolytics-dimension-repository_network_root_nwogithub/codeql
turbo-body-classeslogged-out env-production page-responsive full-width
disable-turbotrue
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release653be32ff5c755082a4db662491bfbbcfc9112c1
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/github/codeql/pull/16458/files#start-of-content
https://github.com/
Sign inhttps://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fgithub%2Fcodeql%2Fpull%2F16458%2Ffiles
GitHubRepository hosting, code review, and collaborationhttps://github.com/platform
CopilotAI coding agent for tasks, projects, and automated workhttps://github.com/features/copilot
ActionsCI/CD and automationhttps://github.com/features/actions
Code QualityQuality analysis, fixes, and enforcementhttps://github.com/features/code-quality
CodespacesCloud development environmentshttps://github.com/features/codespaces
Code SecurityVulnerability detection and remediationhttps://github.com/security/advanced-security/code-security
Secret ProtectionSecret scanning and leak preventionhttps://github.com/security/advanced-security/secret-protection
All Featureshttps://github.com/features
Documentationhttps://docs.github.com/en
Traininghttps://learn.github.com/skills
Certificationshttps://learn.github.com/credentials
Build with Copilothttps://github.com/features/ai
Customize Copilothttps://docs.github.com/en/copilot/how-tos/copilot-on-github/customize-copilot/customize-copilot-overview
Integrationshttps://github.com/marketplace
Trending reposhttps://github.com/trending
GitHub Sponsorshttps://github.com/open-source/sponsors
Developer articleshttps://github.com/resources/articles
ChangelogYour source for the latest features, improvements, and fixes across GitHub products, all in one place.https://github.blog/changelog/
Enterpriseshttps://github.com/enterprise
Teamhttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
See all industrieshttps://github.com/solutions/industry
Why GitHubhttps://github.com/why-github
Customer Storieshttps://github.com/customer-stories
Business insightshttps://github.com/solutions/executive-insights
Trust Centerhttps://github.com/trust-center
GitHub Advanced SecurityEnterprise-grade code securityhttps://github.com/security/advanced-security
Copilot for businessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/enterprise/premium-support
Documentationhttps://docs.github.com/en
Resource centerhttps://github.com/resources
Bloghttps://github.blog/
Events & webinarshttps://github.com/resources/events
Community Forumhttps://github.com/orgs/community/discussions
Customer Storieshttps://github.com/customer-stories
Integrationshttps://github.com/marketplace
Customer supporthttps://help.github.com/
Traininghttps://learn.github.com/skills
GitHub Statushttps://www.githubstatus.com/
Abouthttps://github.com/about
Partnershttps://github.com/partners
Changeloghttps://github.blog/changelog/
GitHub UniverseThe event for developers, by developers — talks, workshops, and what’s next for GitHub.https://githubuniverse.com/
Pricinghttps://github.com/pricing
Sign inhttps://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fgithub%2Fcodeql%2Fpull%2F16458%2Ffiles
Sign uphttps://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fpull_requests%2Fshow%2Ffiles&source=header-repo&source_repo=github%2Fcodeql
Reloadhttps://github.com/github/codeql/pull/16458/files
Reloadhttps://github.com/github/codeql/pull/16458/files
Reloadhttps://github.com/github/codeql/pull/16458/files
githubhttps://github.com/github
codeqlhttps://github.com/github/codeql
Codehttps://github.com/github/codeql
Issues1k (1k)https://github.com/github/codeql/issues
Pull requests516 (516)https://github.com/github/codeql/pulls
Discussionshttps://github.com/github/codeql/discussions
Actionshttps://github.com/github/codeql/actions
Projectshttps://github.com/github/codeql/projects
Security and qualityhttps://github.com/github/codeql/security
Insightshttps://github.com/github/codeql/pulse
Sign up for GitHub https://github.com/signup?return_to=%2Fgithub%2Fcodeql%2Fissues%2Fnew%2Fchoose
terms of servicehttps://docs.github.com/terms
privacy statementhttps://docs.github.com/privacy
Sign inhttps://github.com/login?return_to=%2Fgithub%2Fcodeql%2Fissues%2Fnew%2Fchoose
owen-mchttps://github.com/owen-mc
github:mainhttps://github.com/github/codeql/tree/main
owen-mc:go/fix-mad-for-builtin-functionshttps://github.com/owen-mc/codeql/tree/go/fix-mad-for-builtin-functions
Conversation 0 https://github.com/github/codeql/pull/16458
Commits 5 https://github.com/github/codeql/pull/16458/commits
Checks 0 https://github.com/github/codeql/pull/16458/checks
Findings https://github.com/github/codeql/pull/16458/findings
Files changed https://github.com/github/codeql/pull/16458/files
Please reload this pagehttps://github.com/github/codeql/pull/16458/files
Go: fix hasQualifiedName and models-as-data for built-in functions https://github.com/github/codeql/pull/16458/files#top
Show all changes 5 commits https://github.com/github/codeql/pull/16458/files
fb74a2a Use "" in qualified name for entities without a package owen-mc May 6, 2024 https://github.com/github/codeql/pull/16458/commits/fb74a2a170246090195da7e8fa806a47e9580939
f6f594e parse empty string as package name owen-mc May 6, 2024 https://github.com/github/codeql/pull/16458/commits/f6f594e4b580a7ba50b4591f87fa104502524388
1ccea88 Add change note owen-mc May 8, 2024 https://github.com/github/codeql/pull/16458/commits/1ccea884ff37e261ebea4c9bab34aa6d6f6229a4
57ff30c Update tests: built-in models now work owen-mc May 8, 2024 https://github.com/github/codeql/pull/16458/commits/57ff30c5f3510319313c8521514bf4e502b597a1
4f10cb5 Local flow tests: do not list summary models owen-mc May 8, 2024 https://github.com/github/codeql/pull/16458/commits/4f10cb5fa08688c3f61413a88b9f286f1d3bd9f7
Clear filters https://github.com/github/codeql/pull/16458/files
Please reload this pagehttps://github.com/github/codeql/pull/16458/files
Please reload this pagehttps://github.com/github/codeql/pull/16458/files
2024-05-08-builtin-functions-no-package.md https://github.com/github/codeql/pull/16458/files#diff-aa3704a727fa01be974dede561460406be7883ad303390bbba0fb68a6fb0d0aa
Scopes.qll https://github.com/github/codeql/pull/16458/files#diff-46e38116c1c54c6a7b2ca7cfb2f2ad52edf170531a180630c8a0052e7f49c943
ExternalFlow.qll https://github.com/github/codeql/pull/16458/files#diff-871e92b3cb5f0b3624e585a1f9dbbab7e383d29439bd01d358f0ea072306790e
LocalFlowStep.ql https://github.com/github/codeql/pull/16458/files#diff-4b3cd8df6a18adfcb08f6ff75e52142c7b25d77073c219276cb5e39f4841ea5d
LocalFlowStep.ql https://github.com/github/codeql/pull/16458/files#diff-03f763dab02df77f915b4944e2185af03f9313b7e5b0c64e7d68442cfa855214
CommandInjection.expected https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/blob/main/CODEOWNERS#L10
go/ql/lib/change-notes/2024-05-08-builtin-functions-no-package.mdhttps://github.com/github/codeql/pull/16458/files#diff-aa3704a727fa01be974dede561460406be7883ad303390bbba0fb68a6fb0d0aa
View file https://github.com/owen-mc/codeql/blob/4f10cb5fa08688c3f61413a88b9f286f1d3bd9f7/go/ql/lib/change-notes/2024-05-08-builtin-functions-no-package.md
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/github/codeql/pull/16458/{{ revealButtonHref }}
https://github.com/github/codeql/blob/main/CODEOWNERS#L10
go/ql/lib/semmle/go/Scopes.qllhttps://github.com/github/codeql/pull/16458/files#diff-46e38116c1c54c6a7b2ca7cfb2f2ad52edf170531a180630c8a0052e7f49c943
View file https://github.com/owen-mc/codeql/blob/4f10cb5fa08688c3f61413a88b9f286f1d3bd9f7/go/ql/lib/semmle/go/Scopes.qll
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/github/codeql/pull/16458/{{ revealButtonHref }}
https://github.com/github/codeql/pull/16458/files#diff-46e38116c1c54c6a7b2ca7cfb2f2ad52edf170531a180630c8a0052e7f49c943
https://github.com/github/codeql/pull/16458/files#diff-46e38116c1c54c6a7b2ca7cfb2f2ad52edf170531a180630c8a0052e7f49c943
https://github.com/github/codeql/blob/main/CODEOWNERS#L10
go/ql/lib/semmle/go/dataflow/ExternalFlow.qllhttps://github.com/github/codeql/pull/16458/files#diff-871e92b3cb5f0b3624e585a1f9dbbab7e383d29439bd01d358f0ea072306790e
View file https://github.com/owen-mc/codeql/blob/4f10cb5fa08688c3f61413a88b9f286f1d3bd9f7/go/ql/lib/semmle/go/dataflow/ExternalFlow.qll
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/github/codeql/pull/16458/{{ revealButtonHref }}
https://github.com/github/codeql/pull/16458/files#diff-871e92b3cb5f0b3624e585a1f9dbbab7e383d29439bd01d358f0ea072306790e
https://github.com/github/codeql/pull/16458/files#diff-871e92b3cb5f0b3624e585a1f9dbbab7e383d29439bd01d358f0ea072306790e
https://github.com/github/codeql/blob/main/CODEOWNERS#L10
go/ql/test/library-tests/semmle/go/dataflow/FlowSteps/LocalFlowStep.qlhttps://github.com/github/codeql/pull/16458/files#diff-4b3cd8df6a18adfcb08f6ff75e52142c7b25d77073c219276cb5e39f4841ea5d
View file https://github.com/owen-mc/codeql/blob/4f10cb5fa08688c3f61413a88b9f286f1d3bd9f7/go/ql/test/library-tests/semmle/go/dataflow/FlowSteps/LocalFlowStep.ql
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/github/codeql/pull/16458/{{ revealButtonHref }}
https://github.com/github/codeql/blob/main/CODEOWNERS#L10
go/ql/test/library-tests/semmle/go/dataflow/PromotedFields/LocalFlowStep.qlhttps://github.com/github/codeql/pull/16458/files#diff-03f763dab02df77f915b4944e2185af03f9313b7e5b0c64e7d68442cfa855214
View file https://github.com/owen-mc/codeql/blob/4f10cb5fa08688c3f61413a88b9f286f1d3bd9f7/go/ql/test/library-tests/semmle/go/dataflow/PromotedFields/LocalFlowStep.ql
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/github/codeql/pull/16458/{{ revealButtonHref }}
https://github.com/github/codeql/blob/main/CODEOWNERS#L10
go/ql/test/query-tests/Security/CWE-078/CommandInjection.expectedhttps://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
View file https://github.com/owen-mc/codeql/blob/4f10cb5fa08688c3f61413a88b9f286f1d3bd9f7/go/ql/test/query-tests/Security/CWE-078/CommandInjection.expected
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/github/codeql/pull/16458/{{ revealButtonHref }}
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com/github/codeql/pull/16458/files#diff-cc031605d31e05587fee04c1b1900ef5d4ebaed669fe795a5474f28b97118ea4
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.