Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Hide com.google.protobuf.Any from Swagger introspection #2704

Merged
merged 1 commit into from
Jan 19, 2022
Merged

Hide com.google.protobuf.Any from Swagger introspection #2704

merged 1 commit into from
Jan 19, 2022

Conversation

aravindanr
Copy link
Collaborator

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • Other (please describe):

Changes in this PR

Any type is cpu intensive to introspect and could kill Swagger UI in some cases.

Issue #2694

Alternatives considered

None

@apanicker-nflx apanicker-nflx added the type: maintenance Refactoring label Jan 19, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jan 19, 2022 •
edited
Loading

Unit Test Results

   161 files   - 2     161 suites   - 2   8m 48s ⏱️ -47s
1 130 tests  - 4  1 130 ✔️  - 4  0 💤 ±0  0 ❌ ±0 

Results for commit 95a6835. ± Comparison against base commit 9c932ad.

♻️ This comment has been updated with latest results.

@aravindanr aravindanr added the type: bug bugs/ bug fixes label Jan 19, 2022
@aravindanr aravindanr merged commit 9e6bb58 into main Jan 19, 2022
@aravindanr aravindanr deleted the swagger_fix branch January 19, 2022 06:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jxu-nflx jxu-nflx jxu-nflx approved these changes

@apanicker-nflx apanicker-nflx apanicker-nflx approved these changes

Assignees
No one assigned
Labels
type: bug bugs/ bug fixes type: maintenance Refactoring
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aravindanr @apanicker-nflx @jxu-nflx