Skip to content

feat(ofrep): http handler implementation - #736

Open
ZackarySantana wants to merge 3 commits into
open-feature:mainfrom
ZackarySantana:ofrep-handler
Open

feat(ofrep): http handler implementation#736
ZackarySantana wants to merge 3 commits into
open-feature:mainfrom
ZackarySantana:ofrep-handler

Conversation

@ZackarySantana

Copy link
Copy Markdown

This PR

  • adds an ofrep handler

After starting to adopt an ofrep client, I was surprised there's no minimal http handler wrapper around a go openfeature provider already. This PR adds a very bare bone one. This is my first time contributing to this repo, so feel free to critique it!

How to test

  • adds unit tests for default configuration and with options

@ZackarySantana
ZackarySantana requested review from a team as code owners August 20, 2025 04:38
@ZackarySantana ZackarySantana changed the title feat (ofrep): http handler implementation feat(ofrep): http handler implementation Aug 20, 2025
@thomaspoignant

Copy link
Copy Markdown
Member

Hey @ZackarySantana I am not sure to understand what is the goal of this HTTP Handler.
Can you tell me a bit more about this?

GO has already an OFREP provider https://github.com/open-feature/go-sdk-contrib/tree/main/providers/ofrep, and I am not sure to understand what this is bringing around OFREP.

@ZackarySantana

Copy link
Copy Markdown
Author

Hey @thomaspoignant, this adds a server-side http handler wrapper so a go provider can be exposed over OFREP and then accessed by other clients. For example a provider written in go can be exposed with this handler and then consumed from the javascript ofrep provider (or another go ofrep provider, in a microservice architecture).

@ZackarySantana

Copy link
Copy Markdown
Author

I expose it as a handler and leave serving it to the consumer, so they can add authentication and etc.

Signed-off-by: Zackary Santana <64446617+ZackarySantana@users.noreply.github.com>
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