Skip to content

rpc: Set cache control in the HTTP-RPC response header#9442

Closed
JayT106 wants to merge 19 commits intotendermint:mainfrom
JayT106:main-http-cache
Closed

rpc: Set cache control in the HTTP-RPC response header#9442
JayT106 wants to merge 19 commits intotendermint:mainfrom
JayT106:main-http-cache

Conversation

@JayT106
Copy link
Contributor

@JayT106 JayT106 commented Sep 16, 2022

Cherry-picking PR #6265

PR checklist

  • Tests written/updated, or no tests needed
  • CHANGELOG_PENDING.md updated, or no changelog entry needed
  • Updated relevant documentation (docs/) and code comments, or no
    documentation updates needed

@JayT106 JayT106 requested a review from ebuchman as a code owner September 16, 2022 17:07
@JayT106 JayT106 requested a review from a team September 16, 2022 17:07
@github-actions
Copy link

github-actions bot commented Oct 4, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale for use by stalebot label Oct 4, 2022
@thanethomson thanethomson removed the stale for use by stalebot label Oct 4, 2022
Copy link
Contributor

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

Sorry, I left my comments pending a couple of days ago.
Releasing them now.

JayT106 and others added 2 commits October 25, 2022 02:12
@thanethomson thanethomson added C:rpc Component: JSON RPC, gRPC T:breaking Type: Breaking Change labels Oct 27, 2022
@thanethomson thanethomson mentioned this pull request Oct 31, 2022
3 tasks
@thanethomson
Copy link
Contributor

@JayT106, I've taken over this PR to make a few adjustments in #9650 - please feel free to review the additional changes I've made there and let me know whether this works for you.

@JayT106
Copy link
Contributor Author

JayT106 commented Oct 31, 2022

@JayT106, I've taken over this PR to make a few adjustments in #9650 - please feel free to review the additional changes I've made there and let me know whether this works for you.

sounds good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:rpc Component: JSON RPC, gRPC T:breaking Type: Breaking Change

Projects

Status: Done/Merged

Development

Successfully merging this pull request may close these issues.

4 participants