{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":9113580,"defaultBranch":"master","name":"dcache","ownerLogin":"dCache","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2013-03-30T08:58:49.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/1384620?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1725981846.0","currentOid":""},"activityList":{"items":[{"before":"9c0b160e423c76468a83873adbd5aaaaaadac7e2","after":"b075c04d19864b67829f73016165392a0904f15e","ref":"refs/heads/9.2","pushedAt":"2024-09-20T08:54:02.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mksahakyan","name":null,"path":"/mksahakyan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/14925380?s=80&v=4"},"commit":{"message":"bulk: fix broken command `activities`\n\nMotivation:\nCommit 00ba8c attempted to fix the bulk admin command `activities`, but broke it in a different way. The actual data contains three columns, but four placeholders were specified.\n\nModification:\nRemove the fourth placeholder and fourth column name.\n\nResult:\nThe admin command `activities` in the bulk service works again.\n\nTarget: master\nRequest: 10.0, 10.1\nFixes: #7661\nRequires-notes: yes\nRequires-book: no\nAcked-by: Tigran Mkrtchyan","shortMessageHtmlLink":"bulk: fix broken command <code>activities</code>"}},{"before":"54df6ada6a197c2661bd655648a547b2803810d9","after":"97ace7d80c8dab87de378e975dbd3ee816cf90ba","ref":"refs/heads/10.0","pushedAt":"2024-09-20T08:53:52.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mksahakyan","name":null,"path":"/mksahakyan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/14925380?s=80&v=4"},"commit":{"message":"bulk: fix broken command `activities`\n\nMotivation:\nCommit 00ba8c attempted to fix the bulk admin command `activities`, but broke it in a different way. The actual data contains three columns, but four placeholders were specified.\n\nModification:\nRemove the fourth placeholder and fourth column name.\n\nResult:\nThe admin command `activities` in the bulk service works again.\n\nTarget: master\nRequest: 10.0, 10.1\nFixes: #7661\nRequires-notes: yes\nRequires-book: no\nAcked-by: Tigran Mkrtchyan","shortMessageHtmlLink":"bulk: fix broken command <code>activities</code>"}},{"before":"90199cf135b09305ea5d906f516adfb6e1c5f147","after":"082d2d55c8d583d47f6806a04557917c0e07b683","ref":"refs/heads/10.1","pushedAt":"2024-09-20T08:53:45.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mksahakyan","name":null,"path":"/mksahakyan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/14925380?s=80&v=4"},"commit":{"message":"bulk: fix broken command `activities`\n\nMotivation:\nCommit 00ba8c attempted to fix the bulk admin command `activities`, but broke it in a different way. The actual data contains three columns, but four placeholders were specified.\n\nModification:\nRemove the fourth placeholder and fourth column name.\n\nResult:\nThe admin command `activities` in the bulk service works again.\n\nTarget: master\nRequest: 10.0, 10.1\nFixes: #7661\nRequires-notes: yes\nRequires-book: no\nAcked-by: Tigran Mkrtchyan","shortMessageHtmlLink":"bulk: fix broken command <code>activities</code>"}},{"before":"4e5e01502fc7f1ddd8296fed8eacd86ab46b522d","after":"9fb198954069ecb8954fe41c3f4177cefc00c0ac","ref":"refs/heads/feature/default-migration-dest","pushedAt":"2024-09-18T19:43:55.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"greenc-FNAL","name":"Chris Green","path":"/greenc-FNAL","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/2372949?s=80&v=4"},"commit":{"message":"Address https://rb.dcache.org/r/14316/#comment29388","shortMessageHtmlLink":"Address <a href=\"https://rb.dcache.org/r/14316/#comment29388\" rel=\"nofollow\">https://rb.dcache.org/r/14316/#comment29388</a>"}},{"before":"f1ba73bc3c3c22405693d7e6f8c42e58d64596d6","after":"b6a528ca20107f6037306f16aa7985d4f70749a2","ref":"refs/heads/master","pushedAt":"2024-09-18T17:17:05.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"poolmanager: cancel stage when restore is canceled\n\nMotivation:\nif user cancels the stage request via tape-api, then all in-progress\nstage requests are still processed. This is due to the lack of\nmissing message that can be send from pool manager to a pool.\n\nModification:\n\nAdd (currently just mimic) such message and trigger it when a stage\nrequest is triggered in pool manager.\n\nResult:\nin-the-flight stage request is canceled when pool manager request is\ncanceled.\n\nAcked-by: Albert Rossi\nTarget: master\nRequire-book: no\nRequire-notes: yes","shortMessageHtmlLink":"poolmanager: cancel stage when restore is canceled"}},{"before":"eafa09c2474bfea355d1762bac4d97ed3e54d9ed","after":"4e5e01502fc7f1ddd8296fed8eacd86ab46b522d","ref":"refs/heads/feature/default-migration-dest","pushedAt":"2024-09-17T22:09:32.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"greenc-FNAL","name":"Chris Green","path":"/greenc-FNAL","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/2372949?s=80&v=4"},"commit":{"message":"Move check of number of pool groups returned from server to client\n\nPer https://rb.dcache.org/r/14316/#comment29381\n\nN.B. The MigrationModule itself appears to have no access to the\n`poolList` upon job completion to verify its parameters, so the size\ncheck on the map is currently implemented in\n`PoolListByPoolGroupOfPool.success()`. Please advise if this is\nincorrect.","shortMessageHtmlLink":"Move check of number of pool groups returned from server to client"}},{"before":"55e5e2959792c50fa8a259486f3493144264c778","after":"eafa09c2474bfea355d1762bac4d97ed3e54d9ed","ref":"refs/heads/feature/default-migration-dest","pushedAt":"2024-09-17T22:08:25.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"greenc-FNAL","name":"Chris Green","path":"/greenc-FNAL","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/2372949?s=80&v=4"},"commit":{"message":"Move check of number of pool groups returned from server to client\n\nPer https://rb.dcache.org/r/14316/#comment29381\n\nN.B. The MigrationModule itself appears to have no access to the\n`poolList` upon job completion to verify its parameters, so the size\ncheck on the map is currently implemented in\n`PoolListByPoolGroupOfPool.success()`. Please advise if this is\nincorrect.","shortMessageHtmlLink":"Move check of number of pool groups returned from server to client"}},{"before":"561e3bf2b03826a26dbc9bb9ba1245205fda3429","after":"f1ba73bc3c3c22405693d7e6f8c42e58d64596d6","ref":"refs/heads/master","pushedAt":"2024-09-16T14:04:32.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"ci:workaround oidc-agent bug","shortMessageHtmlLink":"ci:workaround oidc-agent bug"}},{"before":"ebae424a0dafc6c9dc7334fe41e97a52d02826ea","after":"561e3bf2b03826a26dbc9bb9ba1245205fda3429","ref":"refs/heads/master","pushedAt":"2024-09-16T11:09:36.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"pool: include source pool name into `pp ls output`\n\nMotivation:\nWhen pool performs a migration then with `pp ls` admins can see the\nongoing transfers. however, the source pools are not shown.\n\nModification:\nInclude source pool into output of `pp ls` command.\n\nResult:\nBetter observability.\n\nbefore:\n\n```\n[dcache-lab008] (poolA@poolA) admin > \\s poolB pp ls\n124 000014895EDDD7CF44F2AF3055AC89945F34 FSM.Transferring\n```\n\nafter:\n\n```\n[dcache-lab008] (poolB@poolB) admin > \\s poolA pp ls\n101 0000367F9C7C3E6848DA906D4C210FE17076 FSM.Transferring << poolB\n```\n\nAcked-by: Karen Hoyos\nTarget: master\nRequire-book: no\nRequire-notes: yes","shortMessageHtmlLink":"pool: include source pool name into <code>pp ls output</code>"}},{"before":"c165794462fefcb5e74ff20327a9a2f22f1f9ad2","after":"ebae424a0dafc6c9dc7334fe41e97a52d02826ea","ref":"refs/heads/master","pushedAt":"2024-09-13T16:47:04.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"libs: use nfs4j-0.26.0\n\nMotivation:\n\nnew Major version with highlights:\n\nMinor Feature release. Highlights:\n\n - added subs for remote quota service\n - Java 17\n\nChangelog for nfs4j-0.25.0..nfs4j-0.26.0\n* [ad31e60] [maven-release-plugin] prepare for next development iteration\n* [40ccd76] build(deps): bump ch.qos.logback:logback-classic from 1.2.3 to 1.2.13\n* [d167a07] build(deps): bump org.springframework:spring-context\n* [55a5ddc] build(deps): bump com.hazelcast:hazelcast from 5.3.0 to 5.3.5\n* [0ce8d2a] vfs: make file handle immutable class\n* [bd0798b] pom: use java17\n* [02a7c55] proj: add basic server for quick testing\n* [c4cb94d] rquota: add xdrs for rquota protocol\n* [0597235] rquota: add stub for rquota service\n* [c58a552] rquota: add possibility to set quota.\n* [0923c1b] github: add github build action\n* [d744fd4] github: add autopublishing of releases\n* [b9b92db] docs: add nfs42_prot.x nfs_xattr.x rquota.x\n* [37904c9] [maven-release-plugin] prepare branch @{releaseLabel}\n* [1f171e6] [maven-release-plugin] prepare release nfs4j-0.26.0\n\nAcked-by: Marina Sahakyan\nTarget: master\nRequire-book: no\nRequire-notes: yes","shortMessageHtmlLink":"libs: use nfs4j-0.26.0"}},{"before":"f19e1c6a28d9e3d986f3ad4ccd7c3afbd30384ac","after":"c165794462fefcb5e74ff20327a9a2f22f1f9ad2","ref":"refs/heads/master","pushedAt":"2024-09-12T08:53:55.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"ci: split current/old dcache deployments","shortMessageHtmlLink":"ci: split current/old dcache deployments"}},{"before":"d99c031355f87f33175539a5f9d7645d1e95a69f","after":"f19e1c6a28d9e3d986f3ad4ccd7c3afbd30384ac","ref":"refs/heads/master","pushedAt":"2024-09-11T13:07:15.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"Merge pull request #7660 from casmic/typos_min_inst\n\nCorrected a few typos in dcache-minimal-installation.md","shortMessageHtmlLink":"Merge pull request <a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2516772740\" data-permission-text=\"Title is private\" data-url=\"https://github.com/dCache/dcache/issues/7660\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/dCache/dcache/pull/7660/hovercard\" href=\"https://github.com/dCache/dcache/pull/7660\">#7660</a> from casmic/typos_min_inst"}},{"before":"a0ba728fa8cc4ebd4b5915e023658c1b7d9d61c5","after":"d99c031355f87f33175539a5f9d7645d1e95a69f","ref":"refs/heads/master","pushedAt":"2024-09-11T09:45:31.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"bulk: fix broken command `activities`\n\nMotivation:\nCommit 00ba8c attempted to fix the bulk admin command `activities`, but broke it in a different way. The actual data contains three columns, but four placeholders were specified.\n\nModification:\nRemove the fourth placeholder and fourth column name.\n\nResult:\nThe admin command `activities` in the bulk service works again.\n\nTarget: master\nRequest: 10.0, 10.1\nFixes: #7661\nRequires-notes: yes\nRequires-book: no\nAcked-by: Tigran Mkrtchyan","shortMessageHtmlLink":"bulk: fix broken command <code>activities</code>"}},{"before":"da4dc284f95b2009f256e3c051a445b82060b64b","after":"55e5e2959792c50fa8a259486f3493144264c778","ref":"refs/heads/feature/default-migration-dest","pushedAt":"2024-09-10T16:39:53.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"greenc-FNAL","name":"Chris Green","path":"/greenc-FNAL","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/2372949?s=80&v=4"},"commit":{"message":"Fix handling of target list to account for possible omission","shortMessageHtmlLink":"Fix handling of target list to account for possible omission"}},{"before":"8fecee871fa9e0d5c381b5b91806f7cf4749b5d9","after":"9c0b160e423c76468a83873adbd5aaaaaadac7e2","ref":"refs/heads/9.2","pushedAt":"2024-09-10T15:24:07.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"[maven-release-plugin] prepare for next development iteration","shortMessageHtmlLink":"[maven-release-plugin] prepare for next development iteration"}},{"before":"4d14890f7deb0f12696229eb5e23a30a90e1d33e","after":"8fecee871fa9e0d5c381b5b91806f7cf4749b5d9","ref":"refs/heads/9.2","pushedAt":"2024-09-10T15:24:05.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"[maven-release-plugin] prepare release 9.2.25","shortMessageHtmlLink":"[maven-release-plugin] prepare release 9.2.25"}},{"before":"d547d4a58eb87bd6b276a11380c72bc122bfc7af","after":"54df6ada6a197c2661bd655648a547b2803810d9","ref":"refs/heads/10.0","pushedAt":"2024-09-10T15:18:12.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"[maven-release-plugin] prepare for next development iteration","shortMessageHtmlLink":"[maven-release-plugin] prepare for next development iteration"}},{"before":"7a2bfe0a2f2db70a388af1fce23034dcc2877540","after":"d547d4a58eb87bd6b276a11380c72bc122bfc7af","ref":"refs/heads/10.0","pushedAt":"2024-09-10T15:18:09.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"[maven-release-plugin] prepare release 10.0.9","shortMessageHtmlLink":"[maven-release-plugin] prepare release 10.0.9"}},{"before":"dfb446f695555288f663b0c557e8b0bea4e98eb9","after":"90199cf135b09305ea5d906f516adfb6e1c5f147","ref":"refs/heads/10.1","pushedAt":"2024-09-10T15:12:06.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"[maven-release-plugin] prepare for next development iteration","shortMessageHtmlLink":"[maven-release-plugin] prepare for next development iteration"}},{"before":"a89c5034bce36cc50ac874d08d9fca11ba38ce2d","after":"dfb446f695555288f663b0c557e8b0bea4e98eb9","ref":"refs/heads/10.1","pushedAt":"2024-09-10T15:12:03.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"[maven-release-plugin] prepare release 10.1.3","shortMessageHtmlLink":"[maven-release-plugin] prepare release 10.1.3"}},{"before":"729ce8f76a26c565ed1cf8093b3dbf0a9f438b6b","after":"4d14890f7deb0f12696229eb5e23a30a90e1d33e","ref":"refs/heads/9.2","pushedAt":"2024-09-10T13:46:30.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"gplazma: fix broken commit d74d9568167f4\n\njava17 is not allowed\n\n(cherry picked from commit 7a2bfe0a2f2db70a388af1fce23034dcc2877540)\nSigned-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>","shortMessageHtmlLink":"gplazma: fix broken commit <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/dCache/dcache/commit/d74d9568167f40abd8167829271c78252723a4c9/hovercard\" href=\"https://github.com/dCache/dcache/commit/d74d9568167f40abd8167829271c78252723a4c9\"><tt>d74d956</tt></a>"}},{"before":"d74d9568167f40abd8167829271c78252723a4c9","after":"7a2bfe0a2f2db70a388af1fce23034dcc2877540","ref":"refs/heads/10.0","pushedAt":"2024-09-10T13:46:30.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"gplazma: fix broken commit d74d9568167f4\n\njava17 is not allowed","shortMessageHtmlLink":"gplazma: fix broken commit <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/dCache/dcache/commit/d74d9568167f40abd8167829271c78252723a4c9/hovercard\" href=\"https://github.com/dCache/dcache/commit/d74d9568167f40abd8167829271c78252723a4c9\"><tt>d74d956</tt></a>"}},{"before":"83297e934e2d716c7e0dd31b899f425d4c02030c","after":"a0ba728fa8cc4ebd4b5915e023658c1b7d9d61c5","ref":"refs/heads/master","pushedAt":"2024-09-10T13:21:53.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kofemann","name":"Tiramisu Mokka","path":"/kofemann","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/575965?s=80&v=4"},"commit":{"message":"ci: add keycloack and dummy token test","shortMessageHtmlLink":"ci: add keycloack and dummy token test"}},{"before":"5708994ab692c17e5e7fe29277e6cff63bc5071f","after":"729ce8f76a26c565ed1cf8093b3dbf0a9f438b6b","ref":"refs/heads/9.2","pushedAt":"2024-09-10T13:05:11.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"gplazma: multimap fix op regression\n\nMotivation:\n\nCommit ef2dc1e0e5 introduced a regression in the multimap plugin.  Where\nthe 'op' principal type is used, logins will fail with dCache logging\na stacktrace like:\n\n    java.lang.RuntimeException: Failed to create principal: java.lang.NoSuchMethodException: org.dcache.auth.OAuthProviderPrincipal.<init>(java.lang.String)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile$MappablePrincipal.buildPrincipal(GplazmaMultiMapFile.java:148)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile$MappablePrincipal.buildMatcher(GplazmaMultiMapFile.java:163)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.asMatcher(GplazmaMultiMapFile.java:270)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.parseMapFile(GplazmaMultiMapFile.java:248)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.mapping(GplazmaMultiMapFile.java:216)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapPlugin.map(GplazmaMultiMapPlugin.java:37)\n            at org.dcache.gplazma.strategies.DefaultMappingStrategy.lambda$map$0(DefaultMappingStrategy.java:57)\n            at org.dcache.gplazma.strategies.PAMStyleStrategy.callPlugins(PAMStyleStrategy.java:91)\n\nThis problem is because the above commit replaced the single-string\nconstructor that was being used by the multimap plugin via reflection.\n\nModification:\n\nDefine the following semantics:\n\nWhen `op` is used as a principal matcher, the matcher will be selected\nwhen the login has an OAuthProviderPrincipal with that value as its\n(dCache) name; for example, `op:FOO` will match if the login has an\nOAuthProviderPrincipal with name `FOO`.  The URL of the issuer (the\n`iss` claim value) is not considered.  This recreates the previous\nsemantics.\n\nWhen used as a principal, the `op` takes two colon-sparated arguments:\nthe (dCache) name for the OP and the issuer URL (the `iss` claim value).\nFor example, `op:FOO:https://my-op.example.org/` creates an OP with name\n`FOO` and issuer URL `https://my-op.example.org/`.\n\nFor backwards compatibility, if the second colon and the issuer URL is\nomitted then a placeholder URL is used and a warning is logged; for\nexample, `op:FOO` will add a OAuthProviderPrincipal with name `FOO` and\na placeholder issuer URL.\n\nUnit tests are added that verify correct behaviour.\n\nResult:\n\nA regression is fixed in the multimap plugin when `op:` principal type\nis used.\n\nTarget: master\nRequest: 10.1\nRequest: 10.0\nRequest: 9.2\nRequires-notes: yes\nRequires-book: no\nCloses: #7654\nPatch: https://rb.dcache.org/r/14314/\nAcked-by: Tigran Mkrtchyan","shortMessageHtmlLink":"gplazma: multimap fix op regression"}},{"before":"1aff06fe09423386f6700cda59da052308d71fdb","after":"d74d9568167f40abd8167829271c78252723a4c9","ref":"refs/heads/10.0","pushedAt":"2024-09-10T13:05:06.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"gplazma: multimap fix op regression\n\nMotivation:\n\nCommit ef2dc1e0e5 introduced a regression in the multimap plugin.  Where\nthe 'op' principal type is used, logins will fail with dCache logging\na stacktrace like:\n\n    java.lang.RuntimeException: Failed to create principal: java.lang.NoSuchMethodException: org.dcache.auth.OAuthProviderPrincipal.<init>(java.lang.String)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile$MappablePrincipal.buildPrincipal(GplazmaMultiMapFile.java:148)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile$MappablePrincipal.buildMatcher(GplazmaMultiMapFile.java:163)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.asMatcher(GplazmaMultiMapFile.java:270)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.parseMapFile(GplazmaMultiMapFile.java:248)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.mapping(GplazmaMultiMapFile.java:216)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapPlugin.map(GplazmaMultiMapPlugin.java:37)\n            at org.dcache.gplazma.strategies.DefaultMappingStrategy.lambda$map$0(DefaultMappingStrategy.java:57)\n            at org.dcache.gplazma.strategies.PAMStyleStrategy.callPlugins(PAMStyleStrategy.java:91)\n\nThis problem is because the above commit replaced the single-string\nconstructor that was being used by the multimap plugin via reflection.\n\nModification:\n\nDefine the following semantics:\n\nWhen `op` is used as a principal matcher, the matcher will be selected\nwhen the login has an OAuthProviderPrincipal with that value as its\n(dCache) name; for example, `op:FOO` will match if the login has an\nOAuthProviderPrincipal with name `FOO`.  The URL of the issuer (the\n`iss` claim value) is not considered.  This recreates the previous\nsemantics.\n\nWhen used as a principal, the `op` takes two colon-sparated arguments:\nthe (dCache) name for the OP and the issuer URL (the `iss` claim value).\nFor example, `op:FOO:https://my-op.example.org/` creates an OP with name\n`FOO` and issuer URL `https://my-op.example.org/`.\n\nFor backwards compatibility, if the second colon and the issuer URL is\nomitted then a placeholder URL is used and a warning is logged; for\nexample, `op:FOO` will add a OAuthProviderPrincipal with name `FOO` and\na placeholder issuer URL.\n\nUnit tests are added that verify correct behaviour.\n\nResult:\n\nA regression is fixed in the multimap plugin when `op:` principal type\nis used.\n\nTarget: master\nRequest: 10.1\nRequest: 10.0\nRequest: 9.2\nRequires-notes: yes\nRequires-book: no\nCloses: #7654\nPatch: https://rb.dcache.org/r/14314/\nAcked-by: Tigran Mkrtchyan","shortMessageHtmlLink":"gplazma: multimap fix op regression"}},{"before":"c9d8b092d12eab497d437da78e1f54dffc19e2bc","after":"a89c5034bce36cc50ac874d08d9fca11ba38ce2d","ref":"refs/heads/10.1","pushedAt":"2024-09-10T13:05:03.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"gplazma: multimap fix op regression\n\nMotivation:\n\nCommit ef2dc1e0e5 introduced a regression in the multimap plugin.  Where\nthe 'op' principal type is used, logins will fail with dCache logging\na stacktrace like:\n\n    java.lang.RuntimeException: Failed to create principal: java.lang.NoSuchMethodException: org.dcache.auth.OAuthProviderPrincipal.<init>(java.lang.String)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile$MappablePrincipal.buildPrincipal(GplazmaMultiMapFile.java:148)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile$MappablePrincipal.buildMatcher(GplazmaMultiMapFile.java:163)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.asMatcher(GplazmaMultiMapFile.java:270)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.parseMapFile(GplazmaMultiMapFile.java:248)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapFile.mapping(GplazmaMultiMapFile.java:216)\n            at org.dcache.gplazma.plugins.GplazmaMultiMapPlugin.map(GplazmaMultiMapPlugin.java:37)\n            at org.dcache.gplazma.strategies.DefaultMappingStrategy.lambda$map$0(DefaultMappingStrategy.java:57)\n            at org.dcache.gplazma.strategies.PAMStyleStrategy.callPlugins(PAMStyleStrategy.java:91)\n\nThis problem is because the above commit replaced the single-string\nconstructor that was being used by the multimap plugin via reflection.\n\nModification:\n\nDefine the following semantics:\n\nWhen `op` is used as a principal matcher, the matcher will be selected\nwhen the login has an OAuthProviderPrincipal with that value as its\n(dCache) name; for example, `op:FOO` will match if the login has an\nOAuthProviderPrincipal with name `FOO`.  The URL of the issuer (the\n`iss` claim value) is not considered.  This recreates the previous\nsemantics.\n\nWhen used as a principal, the `op` takes two colon-sparated arguments:\nthe (dCache) name for the OP and the issuer URL (the `iss` claim value).\nFor example, `op:FOO:https://my-op.example.org/` creates an OP with name\n`FOO` and issuer URL `https://my-op.example.org/`.\n\nFor backwards compatibility, if the second colon and the issuer URL is\nomitted then a placeholder URL is used and a warning is logged; for\nexample, `op:FOO` will add a OAuthProviderPrincipal with name `FOO` and\na placeholder issuer URL.\n\nUnit tests are added that verify correct behaviour.\n\nResult:\n\nA regression is fixed in the multimap plugin when `op:` principal type\nis used.\n\nTarget: master\nRequest: 10.1\nRequest: 10.0\nRequest: 9.2\nRequires-notes: yes\nRequires-book: no\nCloses: #7654\nPatch: https://rb.dcache.org/r/14314/\nAcked-by: Tigran Mkrtchyan","shortMessageHtmlLink":"gplazma: multimap fix op regression"}},{"before":"4678e1b5aeda157ce41feff175bff76bc3d0a97d","after":"1aff06fe09423386f6700cda59da052308d71fdb","ref":"refs/heads/10.0","pushedAt":"2024-09-10T12:17:10.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"bulk: check targets for empty strings\n\nMotivation:\n\nWhen specifying empty target the bulk proceeds to process\nthe request instead of failing fast.\n\nModification:\n\nFix parsing of target string arguments.\n\nResult:\n\nFail fast with invalid request\n\nPatch: https://rb.dcache.org/r/14312/\nAcked-by: Tigran\nTarget: trunk\nRequest: 10.x, 9.x","shortMessageHtmlLink":"bulk: check targets for empty strings"}},{"before":"b29c463b73ef2c4091b311b5ecc719ed48ad82d6","after":"c9d8b092d12eab497d437da78e1f54dffc19e2bc","ref":"refs/heads/10.1","pushedAt":"2024-09-10T12:17:05.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"bulk: check targets for empty strings\n\nMotivation:\n\nWhen specifying empty target the bulk proceeds to process\nthe request instead of failing fast.\n\nModification:\n\nFix parsing of target string arguments.\n\nResult:\n\nFail fast with invalid request\n\nPatch: https://rb.dcache.org/r/14312/\nAcked-by: Tigran\nTarget: trunk\nRequest: 10.x, 9.x","shortMessageHtmlLink":"bulk: check targets for empty strings"}},{"before":"6043a20af49a69bd66c8e2428b9663ae6355c14a","after":"5708994ab692c17e5e7fe29277e6cff63bc5071f","ref":"refs/heads/9.2","pushedAt":"2024-09-10T12:17:00.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"bulk: check targets for empty strings\n\nMotivation:\n\nWhen specifying empty target the bulk proceeds to process\nthe request instead of failing fast.\n\nModification:\n\nFix parsing of target string arguments.\n\nResult:\n\nFail fast with invalid request\n\nPatch: https://rb.dcache.org/r/14312/\nAcked-by: Tigran\nTarget: trunk\nRequest: 10.x, 9.x","shortMessageHtmlLink":"bulk: check targets for empty strings"}},{"before":"a6fffd406903e87cdde503ae02a6d9df7b2416ff","after":"b29c463b73ef2c4091b311b5ecc719ed48ad82d6","ref":"refs/heads/10.1","pushedAt":"2024-09-10T12:13:34.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lemora","name":"Lea","path":"/lemora","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12768489?s=80&v=4"},"commit":{"message":"gplazma2-oidc-te: fix invalid initialization of plugin\n\nMotivation:\nThe META-INF/gplazma-plugins.xml must point to the correct class name,\notherwise we get:\n\nUnable register new plugin: Class org.dcache.gplazma.plugins.tokenx.TokenExchange could not be found.\n\nModification:\nFix the class name in gplazma-plugins.xml\n\nResult:\nno error on start (and working plugin :) )\n\nAcked-by: Paul Millar\nTarget: master, 10.1\nRequire-book: no\nRequire-notes: yes\n(cherry picked from commit bb836bef2da2ae4ba50d3ea41f87a33f4b58854e)\nSigned-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>","shortMessageHtmlLink":"gplazma2-oidc-te: fix invalid initialization of plugin"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0yMFQwODo1NDowMi4wMDAwMDBazwAAAAS7vfT9","startCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0yMFQwODo1NDowMi4wMDAwMDBazwAAAAS7vfT9","endCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0xMFQxMjoxMzozNC4wMDAwMDBazwAAAASyAu2Z"}},"title":"Activity · dCache/dcache"}