[GitHub] [james-project] chibenwa commented on pull request #1388: JAMES-3756 SessionTranslator should only ask 1 delegation source

2023-01-15 Thread GitBox
chibenwa commented on PR #1388: URL: https://github.com/apache/james-project/pull/1388#issuecomment-1383446688 Ok, just I would need it for some other topics. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [james-project] chibenwa commented on pull request #1388: JAMES-3756 SessionTranslator should only ask 1 delegation source

2023-01-15 Thread GitBox
chibenwa commented on PR #1388: URL: https://github.com/apache/james-project/pull/1388#issuecomment-1383417966 TODO rebase https://github.com/chibenwa/james-project/tree/poc-refacto-delegation (I will do) -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [james-project] chibenwa commented on pull request #1388: JAMES-3756 SessionTranslator should only ask 1 delegation source

2023-01-11 Thread GitBox
chibenwa commented on PR #1388: URL: https://github.com/apache/james-project/pull/1388#issuecomment-1378571262 I have two major issues wit this design: - 1. the naming suck (): too long, not precise - 2. with this change SessionProvider exposes the APIs to bypass its own role