Page MenuHomeSoftware Heritage

keycloak: Fix typo in swh-web client role
ClosedPublic

Authored by anlambert on Tue, Oct 13, 2:33 PM.

Details

Summary

I noticed that the users to which I manually added the swh.web.api.throttling_exempted
permission do not have it anymore since the configuration update in D4211.

Turns out I made a typo in the permission name and thus the permission with correct name
was removed by puppet. My bad, I should have double checked.

14:31 $  bin/octocatalog-diff --octocatalog-diff-args --no-truncate-details -t staging kelvingrove.internal.softwareheritage.org
Found host kelvingrove.internal.softwareheritage.org
Cloning into '/tmp/swh-ocd.xVXOJ5G7/environments/production/data/private'...
done.
Cloning into '/tmp/swh-ocd.xVXOJ5G7/environments/staging/data/private'...
done.
*** Running octocatalog-diff on host kelvingrove.internal.softwareheritage.org
I, [2020-10-13T14:31:44.383601 #2730900]  INFO -- : Catalogs compiled for kelvingrove.internal.softwareheritage.org
I, [2020-10-13T14:31:44.595144 #2730900]  INFO -- : Diffs computed for kelvingrove.internal.softwareheritage.org
diff origin/production/kelvingrove.internal.softwareheritage.org current/kelvingrove.internal.softwareheritage.org
*******************************************
  Keycloak_client[swh-web on SoftwareHeritageStaging] =>
   parameters =>
     roles =>
      - ["swh.web.api.throtlling_exempted", "swh.web.api.graph"]
      + ["swh.web.api.throttling_exempted", "swh.web.api.graph"]
*******************************************
  Keycloak_client[swh-web on SoftwareHeritage] =>
   parameters =>
     roles =>
      - ["swh.web.api.throtlling_exempted", "swh.web.api.graph"]
      + ["swh.web.api.throttling_exempted", "swh.web.api.graph"]
*******************************************
*** End octocatalog-diff on kelvingrove.internal.softwareheritage.org

Diff Detail

Repository
rSPSITE puppet-swh-site
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

anlambert created this revision.Tue, Oct 13, 2:33 PM

lol, i had to read it with pause in between characters to actually see the
double l instead of the double t ¯\_(ツ)_/¯

Thanks

ardumont accepted this revision.Tue, Oct 13, 5:10 PM
This revision is now accepted and ready to land.Tue, Oct 13, 5:10 PM
This revision was automatically updated to reflect the committed changes.