Skip to content

Add custom HTTP header support for proxies#6

Open
markrosan8 wants to merge 1 commit intoopencloud-eu:mainfrom
markrosan8:feature/custom-http-header
Open

Add custom HTTP header support for proxies#6
markrosan8 wants to merge 1 commit intoopencloud-eu:mainfrom
markrosan8:feature/custom-http-header

Conversation

@markrosan8
Copy link
Copy Markdown

Description

Adds support for a user-configurable custom HTTP header that is attached to every outgoing request. The header name and value are read from NSUserDefaults in prepareRequestForScheduling:, the single chokepoint for all HTTP requests in the SDK. Leaving either field empty results in no header being added.

Related Issue

Resolves https://github.com/orgs/opencloud-eu/discussions/2557

Motivation and Context

Users running OpenCloud behind reverse proxies sometimes need a custom HTTP header on all requests for routing or authentication gating. The iOS app currently has no way to attach custom headers, which can prevent the app from working in these deployments.

How Has This Been Tested?

Built and tested on iOS sim. Verified via Traefik access logs that the custom header appears on all outgoing requests. Verified that leaving the fields empty results in no change to existing behavior.

Screenshots (if appropriate):

See companion PR in opencloud-eu/ios.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@guruz guruz self-requested a review April 22, 2026 09:28
@guruz guruz self-assigned this Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants