fix: Removed incorrect string case format from the options.authentica…

[lucasbotingnon]

This PR

Solves a problem with authentication using flipt-web-provider: FliptEvaluationClient using needs to receive options.authentication in camel case, not snake case.

Removed entire if statement from line 92 to line 104. This change will not affect functionality after switching to camelCase, as it will correctly format the auth.

Related Issues

Fixes #1208

How to test

Initialize the provider, and check if the web request has the Auth header:

  const provider = new FliptWebProvider(
    "NAMESPACE",
    {
      url: "URL",
      authentication: {
        clientToken: "123RANDOM_TOKEN321",
      },
    }
  );

  await OpenFeature.setProviderAndWait(provider);

[beeme1mr]

Thanks, looks good to me.

[beeme1mr]

Hey @lucasbotingnon, could you please sign off on your commit. It's a requirement from the CNCF. You can do it by:

1. In your local branch, run: git rebase HEAD~1 --signoff
2. Force push your changes to overwrite the branch: git push --force-with-lease origin bugfix/flip-web-provider-authentication-not-reaching-evaluation-client

Thanks!

[lucasbotingnon]

Hey @lucasbotingnon, could you please sign off on your commit. It's a requirement from the CNCF. You can do it by:

1. In your local branch, run: git rebase HEAD~1 --signoff
2. Force push your changes to overwrite the branch: git push --force-with-lease origin bugfix/flip-web-provider-authentication-not-reaching-evaluation-client

Thanks!

Done! Ty

[markphelps]

thanks @lucasbotingnon !!

[paulopiriquito]

Thanks @lucasbotingnon ! Now we can properly use a secured endpoint for the flipt provider.