[PR #237] [MERGED] Ability to provide headers for a multi-part request #580

New issue

Closed

opened 2026-03-03 16:49:33 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 16:49:33 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/OAuthSwift/OAuthSwift/pull/237
Author: @dparne
Created: 6/17/2016
Status: ✅ Merged
Merged: 7/7/2016
Merged by: @phimage

Base: master ← Head: multipart-request-headers

📝 Commits (3)

c37d110 headers in multipart request
4887e21 Updated all appropriate functions to take in headers as an optional parameter for convenience
a499285 Merge branch 'master' into multipart-request-headers

📊 Changes

2 files changed (+11 additions, -11 deletions)

View changed files

📝 OAuthSwift/OAuth2Swift.swift (+9 -9)
📝 OAuthSwift/OAuthSwiftClient.swift (+2 -2)

📄 Description

Issue faced

The services I am working against are expecting certain headers in the multipart request other than "Content-Type"
Since func postMultiPartRequest is public it would be convenient to be able to pass headers to the underlying request function.

Changes made

Added optional headers parameter and updated all functions affected by it.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/OAuthSwift/OAuthSwift/pull/237 **Author:** [@dparne](https://github.com/dparne) **Created:** 6/17/2016 **Status:** ✅ Merged **Merged:** 7/7/2016 **Merged by:** [@phimage](https://github.com/phimage) **Base:** `master` ← **Head:** `multipart-request-headers` --- ### 📝 Commits (3) - [`c37d110`](https://github.com/OAuthSwift/OAuthSwift/commit/c37d110cbc176c964eec1430921b4c68921d8a0b) headers in multipart request - [`4887e21`](https://github.com/OAuthSwift/OAuthSwift/commit/4887e217e53175b0800db5d9fe6a50f39eb490e2) Updated all appropriate functions to take in headers as an optional parameter for convenience - [`a499285`](https://github.com/OAuthSwift/OAuthSwift/commit/a499285e927e6e59935a59b67504e92648c9fb7f) Merge branch 'master' into multipart-request-headers ### 📊 Changes **2 files changed** (+11 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `OAuthSwift/OAuth2Swift.swift` (+9 -9) 📝 `OAuthSwift/OAuthSwiftClient.swift` (+2 -2) </details> ### 📄 Description ### Issue faced - The services I am working against are expecting certain headers in the multipart request other than "Content-Type" - Since `func postMultiPartRequest` is public it would be convenient to be able to pass headers to the underlying request function. ### Changes made - Added optional headers parameter and updated all functions affected by it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>