paypal/squbs

Webcontext can start with `/`

Open

#340 opened on Dec 2, 2016

View on GitHub
 (2 comments) (1 reaction) (0 assignees)Scala (1,384 stars) (248 forks)batch import
help wanted

Description

The doc currently reads:

The web-context is a string that uniquely identifies the web context of a request to be dispatched to this service. It MUST NOT start with a / character. It can have / characters inside as segment separators in case of multi-segment contexts. And it is allowed to be "" for root context. If multiple services match the request, the longest context match takes precedence.

We should actually allow it to start with a /. Otherwise, it might get annoying while troubleshooting in development.

Contributor guide

Tech stack
None
Domain
documentation
Issue type
documentation
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
familiarity with squbs web context concept
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
95
Research direction
Locate the documentation file describing the web context (likely in the docs folder or README). Modify the text to allow the web context to start with a '/', updating the example and removing the 'MUST NOT start with a /' restriction. Ensure consistency with other parts of the documentation. No code changes are required.