Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client-Side OFREP Provider doesn't type-cast flag values #1059

Open
Chlor87 opened this issue Sep 24, 2024 · 4 comments
Open

Client-Side OFREP Provider doesn't type-cast flag values #1059

Chlor87 opened this issue Sep 24, 2024 · 4 comments

Comments

@Chlor87
Copy link

Chlor87 commented Sep 24, 2024

I'm trying to use Client-Side OFREP Provider with Flipt backend. I noticed that evaluate method doesn't perform any type conversions, just the type assertion. My question is if that's by design or can it be changed? If it can, I can try and post a PR. I also noticed that a native Flipt client performs the type conversions, hence the question.

From what I gathered, the Server-Side OFREP Provider also doesn't perform type conversions.

@beeme1mr
Copy link
Member

Hi @Chlor87, this is by design. Do you know why Flipt needs to cast flag types?

FYI @thomaspoignant @lukas-reining @markphelps

@lukas-reining
Copy link
Member

lukas-reining commented Sep 24, 2024

Do you know why Flipt needs to cast flag types?

@beeme1mr I think even though we have specified this in 1.4.1 [1], we have, with Flipt and Flagsmith, at least two providers that cast flag values that do not match and can be "safely" casted. [2]
I think this is fine as long as this is what the provider considers safely casteable.

For OFREP, as it is more generic, I think this could be hard to do without creating the "JS like" type coercion mess.

@Chlor87
Copy link
Author

Chlor87 commented Sep 25, 2024

Thanks for the swift reponses!
@beeme1mr it appears that Flipt only supports strings and booleans, so in order to have values of other types (numbers and objects), it attempts to do the conversions on the provider side. I'm using the React SDK on top of that which has several utility functions like useNumberFlagValue or useObjectFlagValue which always fail at the moment with OFREP. Also I need to use the remote evaluation to minimize the flag info available to the end user.

@beeme1mr
Copy link
Member

The sdk does type validation so we potentially expose a means to define custom type casting rules. I think this should live in user land and not directly in the provider.

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

No branches or pull requests

3 participants