IdStory Identity Manager

PoC is done here: https://github.com/bcvsolutions/CzechIdMng/commit/a234ab3bdec492d6f6fa3aa20ad06da4ea3d9be1

• Solved problem with show identity-contract. Implemented sort, where more specific routes are first.
• Solved problem with show tree. In this case route does not have a component. Implemented mechanizm for search route with component if the parent component does not exist.

Funny story. I found this part of react-router "react-router-config". Goal of this project is generating Routes by static routes.js. So exactly what we need. I tried to use it, but there are same problems what I already solved and some new. So on the end my work is not useless.

https://github.com/ReactTraining/react-router/tree/master/packages/react-router-config

React router since V4 doesn't parse the query parameters (.../?new=1) :-(.

I implemented parsing in AbstractContextComponent._parseUrlQuery.
Parsing is executed in constructor and result query object is sets to the this.props.location.query (for ensure back compatibility).

I found problem with some definiton of routes. For example:

    {
      path: 'code-lists',
      component: require('./src/content/codelist/CodeLists'),
      access: [ { type: 'HAS_ANY_AUTHORITY', authorities: ['CODELIST_READ'] } ]
    },
    {
      path: 'code-lists/',
      component: require('./src/content/codelist/CodeListRoutes'),
      access: [ { type: 'HAS_ANY_AUTHORITY', authorities: ['CODELIST_READ'] } ],
      childRoutes: [
        {
          path: ':entityId/detail',
          component: require('./src/content/codelist/CodeListDetail'),
          access: [ { type: 'HAS_ANY_AUTHORITY', authorities: ['CODELIST_READ'] } ]
        },
        {
          path: ':entityId/items',
          component: require('./src/content/codelist/CodeListItems'),
          access: [ { type: 'HAS_ANY_AUTHORITY', authorities: ['CODELISTITEM_READ'] } ]
        },
        {
          path: ':entityId/attributes',
          component: require('./src/content/codelist/CodeListAttributes'),
          access: [ { type: 'HAS_ANY_AUTHORITY', authorities: ['FORMATTRIBUTE_READ'] } ]
        }
      ]
    },

I have problem with correct sorting of this routes (from others reasons I have to ignored the slash ). I need to implement some unbender. I need to the second route will be not only code-lists/, but code-lists/:entityId.

React router since V4 doesn't have method onEnter. We used this method for check access rights for the route.

I implemented this check in the AbstractContextComponent. In the place where I generate the Route, I check access rights (and if is module enabled). If not I set to the Route error component (403, 503) or Login. So I don't make a redirect, but I only replacing a component.

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/5601e0f6d77ef38cfaea9e9fc8aa251ad46e9c1b

I implemented method fixerRoute in routes.js.

Solve problem, when route could have more specific path, but this path is defined in every children. Then is problem with correct ordering this routes (since react-router V4). This method try to find this routes and fix it.

For example path of 'code-lists/' is changed to 'code-lists/:entityId'.

- Routing - Disabled module doesn't work - fixed (Commit: https://github.com/bcvsolutions/CzechIdMng/commit/eec70694bc4fd482b8dad93a7832fbb8f9d3e168)

- Routing - Automatic roles for attributes - tabs on detail doesn't work - fixed (Commit: https://github.com/bcvsolutions/CzechIdMng/commit/6bb46cde4a9862fc87e320aaf5ea74dfe5890537)

- Automatic role by attributes - new update request dosen't have prefilled current rules - fixed (Commit: https://github.com/bcvsolutions/CzechIdMng/commit/3242e7dc4453c50da3124291fc7901bc1339c5df)

I agree, the idea simply not use username in url is not sufficient - we need to find another solution and i like yours ... but example with input string:

%%25 - decode(%%25) = %% => encode(%%) = %25%25

Thx for fixes above, i tested it and it works.

I reverted username usage and url encoding:
https://github.com/bcvsolutions/CzechIdMng/commit/aaa72881daed59c1e80de6967984a7ca253a7cd3

Now we need to fix original issue with double encoding :)

Parameters are decoded in routes now.

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/dd2963286537e90e969866d3ed210c3e4d453dc9