You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rocketmq.apache.org by "yyqdbngt (via GitHub)" <gi...@apache.org> on 2026/09/05 13:45:57 UTC

[PR] test(auth): deepen AuthCookie coverage [rocketmq-dashboard]

yyqdbngt opened a new pull request, #3365:
URL: https://github.com/apache/rocketmq-dashboard/pull/3365

   ## Summary
   
   Extends the existing `AuthCookieTest` (3 to 7 tests) to pin down the remaining session-cookie branches.
   
   Added cases:
   - no authorization header and no session cookie yields null;
   - a non-bearer header passes through unchanged when no cookie matches;
   - a custom cookie name is honored on write, and `clear` emits an empty value with `Max-Age=0`;
   - the bearer session-delivery header is matched case-insensitively and absent by default.
   
   ## Why
   
   The helper gates every session-authenticated request; only the cookie flags, header precedence, and cookie fallback were covered before.
   
   ## Testing
   
   `mvn -B test -Dtest=AuthCookieTest` — 7/7 pass; checkstyle (validate) clean.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] test(auth): deepen AuthCookie coverage [rocketmq-dashboard]

Posted by "lizhimins (via GitHub)" <gi...@apache.org>.
lizhimins closed pull request #3365: test(auth): deepen AuthCookie coverage
URL: https://github.com/apache/rocketmq-dashboard/pull/3365


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org