Skip to content

chore(worker): drop x-processing header and always use private data-fair url#107

Open
BatLeDev wants to merge 1 commit into
masterfrom
chore-drop-x-processing-always-private
Open

chore(worker): drop x-processing header and always use private data-fair url#107
BatLeDev wants to merge 1 commit into
masterfrom
chore-drop-x-processing-always-private

Conversation

@BatLeDev

Copy link
Copy Markdown
Member

Simplify how the worker communicates with data-fair.

  • Remove the x-processing private header previously attached to every data-fair request.
  • Remove the getFromPrivateDataFairUrl config flag (env GET_FROM_PRIVATE_DATA_FAIR_URL) and its schema entry.
  • Route all data-fair requests through privateDataFairUrl when it is configured, instead of only write requests (POST/PUT/DELETE/PATCH).

Why: keep all data-fair traffic within internal infrastructure rather than letting read requests leave via the public URL.

Heads-up:

  • Behavior change: GET requests now go through the private URL too. The old code intentionally kept GETs on the public URL so they appeared in metrics — that read-traffic signal goes away when a private URL is set.
  • The GET_FROM_PRIVATE_DATA_FAIR_URL env var no longer exists; remove it from any deployment config that still sets it.

…air url

The x-processing header is no longer consumed: data-fair never read it and
the metrics daemon dropped its tracking in favor of x-client. Remove its
emission and route all data-fair requests through privateDataFairUrl when set
(no more public-url-for-GET-metrics distinction), pruning the now-dead
getFromPrivateDataFairUrl config.
@BatLeDev BatLeDev marked this pull request as ready for review June 26, 2026 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant