Skip to content

Conversation

@Alonexc
Copy link
Contributor

@Alonexc Alonexc commented Aug 15, 2023

Fixes #4360 .

Motivation

fix unSubscribeUrl cannot be null.

Modifications

fix unSubscribeUrl cannot be null.

Documentation

  • Does this pull request introduce a new feature? no

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #4361 (7733019) into master (c29ada6) will not change coverage.
The diff coverage is n/a.

❗ Current head 7733019 differs from pull request most recent head 4aa0561. Consider uploading reports for the commit 4aa0561 to get more accurate results

@@            Coverage Diff            @@
##             master    #4361   +/-   ##
=========================================
  Coverage     17.74%   17.74%           
  Complexity     1502     1502           
=========================================
  Files           600      600           
  Lines         25439    25439           
  Branches       2393     2393           
=========================================
  Hits           4515     4515           
  Misses        20489    20489           
  Partials        435      435           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@mxsm mxsm left a comment

Choose a reason for hiding this comment

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

LGTM

@mxsm mxsm merged commit fe3000c into apache:master Aug 15, 2023
@Alonexc Alonexc deleted the fix_4360 branch August 16, 2023 06:33
xuhongjia pushed a commit to Deckers-Ohana/eventmesh that referenced this pull request Mar 13, 2025
* fix unSubscribeUrl cannot be null

* remove transient and hump naming.
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] http example unSubscribeUrl cannot be null.

3 participants