personium / personium-core

Core module of Personium
https://personium.io
Apache License 2.0
88 stars 16 forks source link

URI returned by response is not URL-encoded. #214

Closed SawamiWataru closed 6 years ago

SawamiWataru commented 6 years ago

Sample

{
  "d": {
    "results": {
      "__metadata": {
        "uri": "https://sample-personium/sample-user-01/__ctl/ExtCell('https://sample-personium/sample-app-00/')",
        "etag": "W/\"1-1524075877220\"",
        "type": "CellCtl.ExtCell"
      },
      "Url": "https://sample-personium/sample-app-00/",
      "__published": "/Date(1524075877220)/",
      "__updated": "/Date(1524075877220)/",
      "_Role": {
        "__deferred": {
          "uri": "https://sample-personium/sample-user-01/__ctl/ExtCell('https://sample-personium/sample-app-00/')/_Role"
        }
      },
      "_Relation": {
        "__deferred": {
          "uri": "https://sample-personium/sample-user-01/__ctl/ExtCell('https://sample-personium/sample-app-00/')/_Relation"
        }
      }
    }
  }
}

Correct URI https://sample-personium/sample-user-01/__ctl/ExtCell('https%3A%2F%2Fsample-personium%2Fsample-app-00%2F')

dixonsiu commented 6 years ago

After the modification, GUI team needs to verify the following sceens.

  1. External Cell List
    image
  2. External Role List
    image
SawamiWataru commented 6 years ago

After the modification, GUI team needs to verify the following sceens.

I confirmed, there was no problem.

SawamiWataru commented 6 years ago

Released v1.6.11.