Skip to content

Conversation

kylejuliandev
Copy link
Contributor

This PR

  • Adds a default placeholder name that gets generated when a Client is generated. This is inspired from the Java SDK (see EventSource.java). The exception is thrown when an optional client name is specified when retrieving a client from Api.Instance. When no name is specified we use a generated one.

Related Issues

Fixes #491

Notes

Follow-up Tasks

How to test

@kylejuliandev kylejuliandev requested a review from a team as a code owner July 2, 2025 19:35
Copy link

codecov bot commented Jul 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.35%. Comparing base (f923cea) to head (f41e0c9).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #508      +/-   ##
==========================================
+ Coverage   87.28%   87.35%   +0.06%     
==========================================
  Files          47       47              
  Lines        1817     1827      +10     
  Branches      189      190       +1     
==========================================
+ Hits         1586     1596      +10     
  Misses        188      188              
  Partials       43       43              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@askpt askpt left a comment

Choose a reason for hiding this comment

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

Just a small remark. I think we should check for whitespace here.

@askpt askpt enabled auto-merge July 3, 2025 14:44
@askpt askpt added this pull request to the merge queue Jul 3, 2025
Merged via the queue into open-feature:main with commit 9151dcd Jul 3, 2025
16 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Jul 3, 2025
🤖 I have created a release *beep* *boop*
---


##
[2.7.0](v2.6.0...v2.7.0)
(2025-07-03)


### 🐛 Bug Fixes

* Add generic to evaluation event builder
([#500](#500))
([68af649](68af649))
* ArgumentNullException when creating a client with optional name
([#508](#508))
([9151dcd](9151dcd))


### ✨ New Features

* Move OTEL hooks to the SDK
([#338](#338))
([77f6e1b](77f6e1b))


### 🧹 Chore

* **deps:** update actions/attest-build-provenance action to v2.4.0
([#495](#495))
([349c073](349c073))
* **deps:** update actions/attest-sbom action to v2.4.0
([#496](#496))
([f7ca416](f7ca416))
* **deps:** update dependency benchmarkdotnet to 0.15.0
([#481](#481))
([714425d](714425d))
* **deps:** update dependency benchmarkdotnet to 0.15.2
([#494](#494))
([cab3807](cab3807))
* **deps:** update dependency microsoft.net.test.sdk to 17.14.0
([#482](#482))
([520d383](520d383))
* **deps:** update dependency microsoft.net.test.sdk to 17.14.1
([#485](#485))
([78bfdbf](78bfdbf))
* **deps:** update dependency opentelemetry.instrumentation.aspnetcore
to 1.12.0
([#505](#505))
([241d880](241d880))
* **deps:** update dependency reqnroll.xunit to 2.4.1
([#483](#483))
([99f7584](99f7584))
* **deps:** update dependency system.valuetuple to 4.6.1
([#503](#503))
([39f884d](39f884d))
* **deps:** update github/codeql-action digest to 39edc49
([#504](#504))
([08ff43c](08ff43c))
* **deps:** update github/codeql-action digest to ce28f5b
([#492](#492))
([cce224f](cce224f))
* **deps:** update github/codeql-action digest to fca7ace
([#486](#486))
([e18ad50](e18ad50))
* **deps:** update opentelemetry-dotnet monorepo to 1.12.0
([#506](#506))
([69dc186](69dc186))
* **deps:** update spec digest to 1965aae
([#499](#499))
([2e3dffd](2e3dffd))
* **deps:** update spec digest to 42340bb
([#493](#493))
([909c51d](909c51d))
* **deps:** update spec digest to c37ac17
([#502](#502))
([38f63fc](38f63fc))
* **deps:** update spec digest to f014806
([#479](#479))
([dbe8b08](dbe8b08))
* fix sample build warning
([#498](#498))
([08a00e1](08a00e1))


### 📚 Documentation

* add XML comment on FeatureClient
([#507](#507))
([f923cea](f923cea))
* updated contributing link on the README
([8435bf7](8435bf7))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Signed-off-by: OpenFeature Bot <[email protected]>
@kylejuliandev kylejuliandev deleted the bug/fix-null-reference-exception-on-adding-handler branch July 3, 2025 16:47
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.

[BUG] ArgumentNullException thrown when adding a Handler to an unnamed IFeatureClient.
3 participants