Project

General

Profile

Bug #4484

Component::toUri escapes ~ instead of +

Added by Davide Pesavento about 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Base
Target version:
Start date:
02/01/2018
Due date:
% Done:

100%

Estimated time:

Description

RFC 3986 defines tilde (~) as an unreserved character and the plus sign (+) as a reserved one.

  reserved    = gen-delims / sub-delims

  gen-delims  = ":" / "/" / "?" / "#" / "[" / "]" / "@"

  sub-delims  = "!" / "$" / "&" / "'" / "(" / ")"
              / "*" / "+" / "," / ";" / "="

  unreserved  = ALPHA / DIGIT / "-" / "." / "_" / "~"

However, the NDN URI encoding escapes (percent-encodes) the tilde but not the plus sign.


Related issues

Blocked by NDN Specifications - Bug #4485: NDN URI spec defines + instead of ~ as unreservedClosed

History

#1 Updated by Junxiao Shi about 1 year ago

  • Blocked by Bug #4485: NDN URI spec defines + instead of ~ as unreserved added

#2 Updated by Junxiao Shi about 1 year ago

This behavior is correct according to NDN Packet Format spec v0.2.1. The spec must be updated before doing anything in ndn-cxx.

#3 Updated by Junxiao Shi about 1 year ago

  • Blocks Feature #4193: Make ASF Strategy less sensitive to timeouts added

#4 Updated by Davide Pesavento about 1 year ago

Junxiao Shi wrote:

This behavior is correct according to NDN Packet Format spec v0.2.1. The spec must be updated before doing anything in ndn-cxx.

The spec is wrong, as I already noted here https://gerrit.named-data.net/4441

#5 Updated by Davide Pesavento about 1 year ago

  • Blocks deleted (Feature #4193: Make ASF Strategy less sensitive to timeouts)

#6 Updated by Davide Pesavento about 1 year ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 50

#7 Updated by Davide Pesavento about 1 year ago

  • Status changed from In Progress to Code review
  • % Done changed from 50 to 100

#8 Updated by Davide Pesavento about 1 year ago

  • Status changed from Code review to Closed

Also available in: Atom PDF