Skip to content

chore: Update cowlib for contract tests to work with OTP 28.#177

Merged
kinyoklion merged 1 commit into
mainfrom
rlamb/update-cowlib-contract-test
Apr 17, 2026
Merged

chore: Update cowlib for contract tests to work with OTP 28.#177
kinyoklion merged 1 commit into
mainfrom
rlamb/update-cowlib-contract-test

Conversation

@kinyoklion

@kinyoklion kinyoklion commented Apr 16, 2026

Copy link
Copy Markdown
Member

Note

Medium Risk
Dependency upgrade of the HTTP server library plus overridden dependency resolution could introduce subtle runtime or build differences in the test service, especially around Cowboy/Ranch integration.

Overview
Updates test-service to use cowboy 2.14.2 (from 2.8.0) to improve compatibility with newer OTP versions.

Adds a rebar.config overrides block that forces cowboy to resolve with cowlib and ranch, as a workaround for ninenines/cowboy#1670 and older rebar3 behavior.

Reviewed by Cursor Bugbot for commit c3a1e03. Bugbot is set up for automated code reviews on this repo. Configure here.

@kinyoklion kinyoklion requested a review from a team as a code owner April 16, 2026 22:04
Comment thread test-service/rebar.config
ldclient
]}.

%% Overridden for: https://github.com/ninenines/cowboy/issues/1670

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I wouldn't want to do this in the SDK, but for the contract tests I think unconstraining this is likely fine.

@kinyoklion kinyoklion merged commit c43a63c into main Apr 17, 2026
12 checks passed
@kinyoklion kinyoklion deleted the rlamb/update-cowlib-contract-test branch April 17, 2026 15:36
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