docs: add ssePort, protocol config, and port mapping documentation#806
Open
officialasishkumar wants to merge 2 commits intomainfrom
Open
docs: add ssePort, protocol config, and port mapping documentation#806officialasishkumar wants to merge 2 commits intomainfrom
officialasishkumar wants to merge 2 commits intomainfrom
Conversation
Document the new ssePort field, extensible protocol-level port overrides, and replaceWith port mapping. Update the replay target resolution order to reflect the new priority layers.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Incremental Review (since 3db12a5)The incremental changes address both previous review comments:
The documentation is now internally consistent:
Files Reviewed (1 file)
Reviewed by claude-4.5-opus-20251124 · 94,993 tokens |
There was a problem hiding this comment.
Pull request overview
Updates the Keploy v4 configuration-file documentation to better explain replay host/port overriding, including new protocol-specific port overrides, an ssePort option, and replaceWith port-to-port mappings, plus a clarified (and expanded) target resolution order.
Changes:
- Documents new
test.ssePortandtest.protocol.<protocol>.portoptions with YAML examples. - Extends
replaceWithdocs to includeportmappings in addition to URL / gRPC:authorityreplacements. - Revises the “Replay Target Resolution Order” section to reflect the new override precedence.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
versioned_docs/version-4.0.0/running-keploy/configuration-file.md
Outdated
Show resolved
Hide resolved
versioned_docs/version-4.0.0/running-keploy/configuration-file.md
Outdated
Show resolved
Hide resolved
- Remove CLI flag reference from ssePort description - Fix Replay Target Resolution Order: swap steps 6 and 7 so replaceWith matching comes before test.host fallback
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What has changed?
This pull request updates the documentation for the Keploy configuration file to clarify and extend how port and host overrides are handled during test replay. It introduces new configuration options for protocol-specific port overrides, documents the new
ssePortfield, and revises the target resolution order for improved clarity and flexibility.Configuration options and protocol-specific overrides:
protocolsection in thetestconfiguration, allowing users to specify custom ports forhttp,sse, andgrpcprotocols. This makes it easier to add new protocols in the future without code changes. (versioned_docs/version-4.0.0/running-keploy/configuration-file.md) [1] [2]ssePortfield, which allows users to set a custom port for SSE (Server-Sent Events) test cases. This field overrides the general HTTP port for SSE requests. (versioned_docs/version-4.0.0/running-keploy/configuration-file.md) [1] [2]Replay target resolution and replaceWith enhancements:
versioned_docs/version-4.0.0/running-keploy/configuration-file.md)replaceWithfeature to support port-to-port mappings in addition to URL/gRPC authority replacements, and documented how these mappings are applied last as the highest-priority override. (versioned_docs/version-4.0.0/running-keploy/configuration-file.md) [1] [2]Documentation improvements:
versioned_docs/version-4.0.0/running-keploy/configuration-file.md)This PR Resolves keploy/keploy#3974
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please run npm run build and npm run serve to check if the changes are working as expected. Please include screenshots of the output of both the commands. Add screenshots/gif of the changes if possible.
Checklist: