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

Add Endpoint service dependency after new CORS filter #3293

Merged
merged 1 commit into from
Apr 16, 2021

Conversation

lorthirk
Copy link

After merging #3278, the Endpoint service dependency is needed also in JobEngine Application due to the new CORS Filter

Related Issue
No related issues

@lorthirk lorthirk added the Bug This is a bug or an unexpected behaviour. Fix it! label Apr 15, 2021
@lorthirk lorthirk requested a review from Coduz April 15, 2021 10:43
@codecov
Copy link

codecov bot commented Apr 15, 2021

Codecov Report

Merging #3293 (0cd8928) into develop (c345545) will decrease coverage by 16.69%.
The diff coverage is n/a.

❗ Current head 0cd8928 differs from pull request most recent head 0ea53d1. Consider uploading reports for the commit 0ea53d1 to get more accurate results
Impacted file tree graph

@@              Coverage Diff               @@
##             develop    #3293       +/-   ##
==============================================
- Coverage      54.13%   37.43%   -16.70%     
+ Complexity       787      689       -98     
==============================================
  Files           1475     1475               
  Lines          30150    30150               
  Branches        2520     2520               
==============================================
- Hits           16322    11287     -5035     
- Misses         12909    17964     +5055     
+ Partials         919      899       -20     
Impacted Files Coverage Δ Complexity Δ
...rc/main/java/org/eclipse/kapua/KapuaException.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...c/main/java/org/eclipse/kapua/KapuaErrorCodes.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...in/java/org/eclipse/kapua/broker/BrokerDomain.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...ain/java/org/eclipse/kapua/event/ServiceEvent.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...n/java/org/eclipse/kapua/broker/BrokerDomains.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...va/org/eclipse/kapua/commons/util/SystemUtils.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...n/java/org/eclipse/kapua/model/domain/Actions.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
.../java/org/eclipse/kapua/message/KapuaPosition.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
.../java/org/eclipse/kapua/KapuaRuntimeException.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
.../java/org/eclipse/kapua/model/query/SortOrder.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
... and 438 more

@lorthirk lorthirk force-pushed the fix-corsFilterJobEngine branch 2 times, most recently from ccf65e5 to c04f406 Compare April 15, 2021 11:29
Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
@lorthirk lorthirk merged commit 58f80bf into eclipse:develop Apr 16, 2021
@lorthirk lorthirk deleted the fix-corsFilterJobEngine branch April 16, 2021 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug or an unexpected behaviour. Fix it!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants