Add onUpgrade method to WebSocket RequestLogger

[Copilot]

• Analyze review comments from @tipsy
• Remove unnecessary getter method from WsConfig.java (lines 77-82) - made field public instead
• Fix comment in WsConfig.java to say "after" instead of "before"
• Update 404 case to use empty string for matchedPath instead of requestUri (use default args)
• Use default arguments in JavalinWsServletContext constructor to simplify usage
• Extract timing calculation to a reusable function (calculateExecutionTimeMs)
• Test the changes with build and tests - all WebSocket tests passing
• Reply to review comments

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[tipsy]

Don't think we need this.

[tipsy]

Isn't it after? 🤔

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.39%. Comparing base (e359798) to head (55b0504).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2450      +/-   ##
============================================
+ Coverage     86.26%   86.39%   +0.13%     
- Complexity     1318     1435     +117     
============================================
  Files           148      148              
  Lines          4506     4587      +81     
  Branches        514      490      -24     
============================================
+ Hits           3887     3963      +76     
- Misses          400      407       +7     
+ Partials        219      217       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[tipsy]

This should probably be null, since it didn't match a path.

[tipsy]

Maybe these empty maps could be default args, so the whole statement could be val upgradeContext = JavalinWsServletContext(servletContextConfig, req, res)

[tipsy]

Could we pull this out to a function? We are using it somewhere else too.

[tipsy]

@copilot please update your PR based on the review comments.

[Copilot]

@copilot please update your PR based on the review comments.

I've addressed all the review feedback:

1. Removed getter method - Made wsUpgradeLogger field public instead for cross-package access
2. Fixed timing comment - Changed "before" to "after" the upgrade process
3. Fixed 404 matchedPath - Now uses empty string (default arg) instead of requestUri
4. Simplified constructor calls - Added default arguments to JavalinWsServletContext
5. Extracted timing calculation - Created reusable calculateExecutionTimeMs() function

All changes tested and WebSocket tests passing. Commit: 55b0504