Skip to content

Updated Detect AI workflow use pull_request_target - #2462

Merged
tomkralidis merged 1 commit into
geopython:masterfrom
doublebyte1:fix-workflow
Oct 21, 2026
Merged

tomkralidis merged 1 commit into
geopython:masterfrom
doublebyte1:fix-workflow

Conversation

@doublebyte1

@doublebyte1 doublebyte1 commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Overview

Workflows triggered by Pull Requests from forks to the upstream repository don't have access to secrets for security reasons. However, workflows triggered by pull_request_target can access secrets. To make them secure, we need to put some safeguards in place by hardening the restrictions and making sure they never checkout or run the PR code. In this case we only access the PR url, to pass it as an argument to the classifier script.

Related Issue / discussion

#2461

Additional information

https://docs.github.com/en/actions/reference/security/securely-using-pull_request_target

Dependency policy (RFC2)

  • I have ensured that this PR meets RFC2 requirements

Updates to public demo

Contributions and licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution
  • I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

@tomkralidis tomkralidis added this to the 0.25.0 milestone Oct 21, 2026
@tomkralidis
tomkralidis self-requested a review October 21, 2026 14:38
@tomkralidis tomkralidis added the enhancement New feature or request label Oct 21, 2026
@tomkralidis
tomkralidis merged commit 65908b1 into geopython:master Oct 21, 2026
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants