Skip to content

PoC: Support for both old and new schemas#1

Closed
bacherfl wants to merge 3 commits intomainfrom
poc/use-renamed-schemas
Closed

PoC: Support for both old and new schemas#1
bacherfl wants to merge 3 commits intomainfrom
poc/use-renamed-schemas

Conversation

@bacherfl
Copy link
Owner

This PR aims to outline how to both support the old and new schemas. See open-feature#948 for reference

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
}

func (b bufSwitchHandler) ServeHTTP(writer http.ResponseWriter, request *http.Request) {
if strings.HasPrefix(request.URL.Path, "/flagd") {

Choose a reason for hiding this comment

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

simple enough!

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@Kavindu-Dodan
Copy link

@bacherfl nice work.

I think we should aim to push this out to all of flagd components. I prefer the new schema namespaces over the old ones.

@bacherfl bacherfl closed this May 21, 2024
@bacherfl bacherfl deleted the poc/use-renamed-schemas branch May 21, 2024 10:38
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.

3 participants