Project

General

Profile

Actions
Copy link

Bug #4484

closed

Component::toUri escapes ~ instead of +

Added by Davide Pesavento almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Davide Pesavento
Category:
Base
Target version:
v0.7
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 1 (0 open — 1 closed)

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

Actions
Actions
Copy link
 #1

Updated by Junxiao Shi almost 7 years ago

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

Updated by Junxiao Shi almost 7 years 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.

Actions
Copy link
 #3

Updated by Junxiao Shi almost 7 years ago

  • Blocks Feature #4193: Make ASF strategy less sensitive to timeouts added
Actions
Copy link
 #4

Updated by Davide Pesavento almost 7 years 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

Actions
Copy link
 #5

Updated by Davide Pesavento almost 7 years ago

  • Blocks deleted (Feature #4193: Make ASF strategy less sensitive to timeouts)
Actions
Copy link
 #6

Updated by Davide Pesavento almost 7 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 50
Actions
Copy link
 #7

Updated by Davide Pesavento almost 7 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 50 to 100
Actions
Copy link
 #8

Updated by Davide Pesavento over 6 years ago

  • Status changed from Code review to Closed
Actions
Copy link

Also available in: Atom PDF