Skip to content
This repository was archived by the owner on Jun 30, 2023. It is now read-only.

Allow to override EndpointsMethodHandler creation in EndpointsServlet#160

Merged
tangiel merged 1 commit intocloudendpoints:masterfrom
AODocs:method_handler_hook
Aug 16, 2018
Merged

Allow to override EndpointsMethodHandler creation in EndpointsServlet#160
tangiel merged 1 commit intocloudendpoints:masterfrom
AODocs:method_handler_hook

Conversation

@clementdenis
Copy link
Contributor

This very small change would allow to customize serialization mechanisms and other Endpoints internals that are currently hidden.

- This allows full customization of response serialization, etc.
@codecov-io
Copy link

Codecov Report

Merging #160 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #160      +/-   ##
============================================
+ Coverage        80%   80.02%   +0.02%     
- Complexity     1680     1682       +2     
============================================
  Files           156      156              
  Lines          5597     5598       +1     
  Branches        731      731              
============================================
+ Hits           4478     4480       +2     
  Misses          839      839              
+ Partials        280      279       -1
Impacted Files Coverage Δ Complexity Δ
...va/com/google/api/server/spi/EndpointsServlet.java 89.83% <100%> (+0.17%) 18 <1> (+1) ⬆️
.../server/spi/discovery/CommonPathPrefixBuilder.java 100% <0%> (+5.55%) 9% <0%> (+1%) ⬆️

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 c2a26bd...594d87c. Read the comment docs.

@tangiel tangiel merged commit 2f34aad into cloudendpoints:master Aug 16, 2018
@clementdenis clementdenis deleted the method_handler_hook branch February 21, 2022 13:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants