Skip to content

Commit 79aecf8

Browse files
authored
Merge pull request #15 from openstandia/fix-content-type
fix: use lowercase 't' in Content-type header to prevent GitHub API override
2 parents 0e38eeb + c9f065b commit 79aecf8

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/main/java/org/kohsuke/github/SCIMConstants.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,9 @@ public final class SCIMConstants {
1111
public static final String SCIM_ACCEPT = "application/scim+json";
1212
public static final String GITHUB_API_VERSION = "2022-11-28";
1313

14-
public static final String HEADER_CONTENT_TYPE = "Content-Type";
14+
// Be careful, as it will be overwritten by the following if we use Content-Type.
15+
// https://github.com/hub4j/github-api/blob/github-api-1.122/src/main/java/org/kohsuke/github/GitHubHttpUrlConnectionClient.java#L141-L142
16+
public static final String HEADER_CONTENT_TYPE = "Content-type";
1517
public static final String HEADER_ACCEPT = "Accept";
1618
public static final String HEADER_API_VERSION = "X-GitHub-Api-Version";
1719

0 commit comments

Comments
 (0)