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

Do not discard the root exception when things go wrong #1743

Closed
wants to merge 1 commit into from

Conversation

aclement
Copy link
Contributor

The root exception can detail an important root cause. For
example with native-images it can indicate a flag you need
to set for the native-image to compile correctly:
Caused by: java.net.MalformedURLException:
Accessing an URL protocol that was not enabled. The URL
protocol http is supported but not enabled by default.
It must be enabled by adding the -H:EnableURLProtocols=http
option to the native-image command.

The root exception can detail an important root cause. For
example with native-images it can indicate a flag you need
to set for the native-image to compile correctly:
Caused by: java.net.MalformedURLException:
  Accessing an URL protocol that was not enabled. The URL
  protocol http is supported but not enabled by default.
  It must be enabled by adding the -H:EnableURLProtocols=http
  option to the native-image command.
@codecov
Copy link

codecov bot commented Nov 10, 2020

Codecov Report

Merging #1743 (b56819d) into master (c25e3d6) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1743   +/-   ##
=========================================
  Coverage     77.31%   77.31%           
  Complexity     1250     1250           
=========================================
  Files           160      160           
  Lines          4610     4610           
  Branches        615      615           
=========================================
  Hits           3564     3564           
  Misses          809      809           
  Partials        237      237           
Impacted Files Coverage Δ Complexity Δ
...rk/cloud/config/client/ConfigClientProperties.java 91.26% <0.00%> (ø) 47.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c25e3d6...b56819d. Read the comment docs.

@spencergibb spencergibb added this to In progress in 2020.0.0-M5 via automation Nov 10, 2020
@spencergibb spencergibb added this to In progress in Hoxton.SR10 via automation Nov 10, 2020
@spencergibb
Copy link
Member

Cherry-picked to 2.2.x and merged forward via bf2c61f

2020.0.0-M5 automation moved this from In progress to Done Nov 10, 2020
Hoxton.SR10 automation moved this from In progress to Done Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
2020.0.0-M5
  
Done
Hoxton.SR10
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants