Skip to content

feat: add flagd provider implementation #66

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Aug 31, 2022
Merged

feat: add flagd provider implementation #66

merged 6 commits into from
Aug 31, 2022

Conversation

toddbaert
Copy link
Member

@toddbaert toddbaert commented Aug 17, 2022 •
edited
Loading

  • adding grpc generation for flagd communication (git submodule pull of schemas repo in maven exec plugin)
  • provider implementation
  • tests

@toddbaert toddbaert force-pushed the feat/flagd-communication branch from d07e4e4 to a711919 Compare August 17, 2022 20:39
@toddbaert toddbaert marked this pull request as draft August 17, 2022 20:39
@beeme1mr beeme1mr linked an issue Aug 25, 2022 that may be closed by this pull request
flagD Java Provider #68
Closed
@toddbaert toddbaert force-pushed the feat/flagd-communication branch 8 times, most recently from 953a035 to 27c08d1 Compare August 31, 2022 01:38
@toddbaert toddbaert marked this pull request as ready for review August 31, 2022 01:38
@toddbaert toddbaert force-pushed the feat/flagd-communication branch from 27c08d1 to 48b61f5 Compare August 31, 2022 01:40
@toddbaert toddbaert requested a review from skyerus August 31, 2022 01:41
justinabrahms
justinabrahms reviewed Aug 31, 2022
View reviewed changes
Copy link
Member

@justinabrahms justinabrahms left a comment

Choose a reason for hiding this comment

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

Tbh, this didn't look fun to write. :-/

@toddbaert
Copy link
Member Author

@justinabrahms

Tbh, this didn't look fun to write. :-/

It wasn't at first, but the SDK changes around the JSON values made it SO MUCH BETTER, I said as much to @beeme1mr earlier. Dealing with the generated protobuf code is a bit tedious, but overall it saves you from writing a ridiculous amount of boilderplate. If you look in target/generated-sources/ you can see how much java is generated from the protobuf spec.

@toddbaert toddbaert force-pushed the feat/flagd-communication branch 3 times, most recently from b8512bb to 3fb7bb3 Compare August 31, 2022 17:04
@toddbaert toddbaert requested a review from beeme1mr August 31, 2022 17:27
@toddbaert toddbaert force-pushed the feat/flagd-communication branch from 740a1fa to d1ba7df Compare August 31, 2022 18:54
@toddbaert
Make protocol enum
235b158 
Signed-off-by: Todd Baert <[email protected]>
@toddbaert
use provided scope for javasdk
701c380 
Signed-off-by: Todd Baert <[email protected]>
@toddbaert
Extract defaults to vars
254dc71 
Signed-off-by: Todd Baert <[email protected]>
@toddbaert toddbaert force-pushed the feat/flagd-communication branch from 845c30e to 254dc71 Compare August 31, 2022 19:40
@toddbaert toddbaert merged commit 8d299c4 into main Aug 31, 2022
@github-actions github-actions bot mentioned this pull request Aug 31, 2022
Merged
@beeme1mr beeme1mr deleted the feat/flagd-communication branch September 2, 2022 13:18
This was referenced Sep 16, 2022
Merged
Closed
toddbaert added a commit to toddbaert/java-sdk-contrib that referenced this pull request Sep 26, 2022
@james-milligan @beeme1mr
@thomaspoignant @toddbaert
feat: flagd provider (open-feature#66)
9d6cb86 
* initial commit

Signed-off-by: James-Milligan <[email protected]>

* nx format:write

Signed-off-by: James-Milligan <[email protected]>

* udpated nxignore to exclude generated files

Signed-off-by: James-Milligan <[email protected]>

* fixed linting issues

Signed-off-by: James-Milligan <[email protected]>

* typo

Signed-off-by: James-Milligan <[email protected]>

* removed nested .vscode

Signed-off-by: James-Milligan <[email protected]>

* added typing to axios response objects

Signed-off-by: James-Milligan <[email protected]>

* removed proto files from version control

Signed-off-by: James-Milligan <[email protected]>

* required commit for nx run-command

Signed-off-by: James-Milligan <[email protected]>

* actions

Signed-off-by: James-Milligan <[email protected]>

* dependency updates + removal of cantextTransformer method

Signed-off-by: James-Milligan <[email protected]>

* removed forked submodule

Signed-off-by: James-Milligan <[email protected]>

* switched to open-feature/schemas

Signed-off-by: James-Milligan <[email protected]>

* dependency sync

Signed-off-by: James-Milligan <[email protected]>

* merge fixes

Signed-off-by: James-Milligan <[email protected]>

* workspace fixes

Signed-off-by: James-Milligan <[email protected]>

* merge fixes

Signed-off-by: James-Milligan <[email protected]>

* merge fixes

Signed-off-by: James-Milligan <[email protected]>

* merge fixes - added go to ci

Signed-off-by: James-Milligan <[email protected]>

* merge fixes

Signed-off-by: James-Milligan <[email protected]>

* updated go version in github action setup

Signed-off-by: James-Milligan <[email protected]>

* squashed commits

Signed-off-by: James-Milligan <[email protected]>

* buf setup test => avoid using go install buf

Signed-off-by: James-Milligan <[email protected]>

* removed go setup + removed submodule issue => error

Signed-off-by: James-Milligan <[email protected]>

* removed go setup + removed submodule issue => error

Signed-off-by: James-Milligan <[email protected]>

* happy path unit tests

Signed-off-by: James-Milligan <[email protected]>

* more unit tests + fixes

Signed-off-by: James-Milligan <[email protected]>

* set noImplicitOverride to false

Signed-off-by: James-Milligan <[email protected]>

* errorCode consolidation

Signed-off-by: James-Milligan <[email protected]>

* removed tslint ignores and .vscode files

Signed-off-by: James-Milligan <[email protected]>

* package.json cleanup

Signed-off-by: James-Milligan <[email protected]>

* Update project.json

Signed-off-by: James Milligan <[email protected]>

* Apply suggestions from code review

Co-authored-by: Michael Beemer <[email protected]>
Signed-off-by: James Milligan <[email protected]>

* PR updates

Signed-off-by: James-Milligan <[email protected]>

* removed makefile

Signed-off-by: James-Milligan <[email protected]>

* replaced hardcoded errorCode and reason values with imported values from nodesdk

Signed-off-by: James-Milligan <[email protected]>

* style updates, simplification of provider construction, README updates

Signed-off-by: James-Milligan <[email protected]>

* removed variant value "default_value" on error/default, simplified checkResponse function

Signed-off-by: James-Milligan <[email protected]>

* Update README.md

Signed-off-by: James Milligan <[email protected]>

* Update libs/providers/flagd/package.json

Co-authored-by: Thomas Poignant <[email protected]>
Signed-off-by: James Milligan <[email protected]>

* added encodeURI to urls in http service

Signed-off-by: James-Milligan <[email protected]>

* Update pushlish script

Signed-off-by: Todd Baert <[email protected]>

* encodeURIComponent(flagKey) in http service

Signed-off-by: James-Milligan <[email protected]>

* added protocol option

Signed-off-by: James-Milligan <[email protected]>

* update README

Signed-off-by: James-Milligan <[email protected]>

Co-authored-by: Michael Beemer <[email protected]>
Co-authored-by: Thomas Poignant <[email protected]>
Co-authored-by: Todd Baert <[email protected]>
DBlanchard88 pushed a commit to DBlanchard88/java-sdk-contrib that referenced this pull request Apr 29, 2024
@toddbaert
chore: Add release plz (open-feature#66)
7e38e54 
* Add release plz

Signed-off-by: Todd Baert <[email protected]>

* Update release doc

Signed-off-by: Todd Baert <[email protected]>

* Add notes about adding release content

Signed-off-by: Todd Baert <[email protected]>

Signed-off-by: Todd Baert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@justinabrahms justinabrahms justinabrahms approved these changes

@beeme1mr beeme1mr beeme1mr approved these changes

@james-milligan james-milligan Awaiting requested review from james-milligan

@skyerus skyerus Awaiting requested review from skyerus

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

flagD Java Provider
3 participants
@toddbaert @justinabrahms @beeme1mr