Skip to content

fix OAProc subscriber handling - #2479

Merged
tomkralidis merged 2 commits into
masterfrom
oaproc-subscriber-fix
Nov 6, 2026
Merged

tomkralidis merged 2 commits into
masterfrom
oaproc-subscriber-fix

Conversation

@tomkralidis

Copy link
Copy Markdown
Member

Overview

This PR fixes a race condition by keeping the processor as a local definition (vs. class property).

Related Issue / discussion

Additional information

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 Nov 4, 2026
@tomkralidis tomkralidis added bug Something isn't working OGC API - Processes OGC API - Processes labels Nov 4, 2026
@github-actions github-actions Bot added the Human generated Label for the detect-ai workflow classifier label Nov 4, 2026
@tomkralidis
tomkralidis requested a review from a team November 4, 2026 23:50

@francbartoli francbartoli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is worth adding a deterministic regression tests using processors with different allow_internal_requests settings

@tomkralidis

Copy link
Copy Markdown
Member Author

It is worth adding a deterministic regression tests using processors with different allow_internal_requests settings

Tests added in 7511f2a

@tomkralidis
tomkralidis merged commit 133f077 into master Nov 6, 2026
10 of 12 checks passed
@tomkralidis
tomkralidis deleted the oaproc-subscriber-fix branch November 6, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Human generated Label for the detect-ai workflow classifier OGC API - Processes OGC API - Processes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants