feat(hearings): add waiting room and admission controls - #1652
Open
bhavyajain0810 wants to merge 2 commits into
Open
feat(hearings): add waiting room and admission controls#1652bhavyajain0810 wants to merge 2 commits into
bhavyajain0810 wants to merge 2 commits into
Conversation
|
@bhavyajain0810 is attempting to deploy a commit to the CodeBlooded's projects Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
|
Hi @bhavyajain0810, thanks for contributing to Nyay Setu! 🎉 I have automatically:
Our workflows will now analyze your changes to classify:
Tip Ensure your PR description references the issue it resolves (e.g. Happy coding! 🚀 |
Contributor
Author
|
Please review the PR #1652 for issue #1628 CI updateThe The remaining failures appear to be unrelated to this PR:
I have not changed any unrelated files just to bypass these checks. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR introduces an application-level waiting room and admission-control system for virtual court hearings.
Previously, participants could obtain or construct the Jitsi room URL and enter a hearing without waiting for the judge. This change ensures that the room identifier is disclosed only after the authenticated participant has requested access and the assigned judge has admitted them.
Changes included
Added participant admission states:
INVITEDWAITINGADMITTEDREJECTEDEXPIREDAdded secure, unique participant access tokens:
X-Hearing-Access-Tokenrequest headerAdded judge-controlled hearing operations:
Prevented hearing room identifiers from being exposed through regular hearing-list and hearing-detail responses.
Disabled the legacy direct-join endpoint.
Added participant waiting-room polling for litigants and lawyers.
Updated legacy judge hearing entry points to redirect to the secured Conduct Hearing page.
Added database migration
V54__add_virtual_hearing_waiting_room.sql.Added service and controller tests for:
Security scope
This implementation provides application-level waiting-room protection and prevents NyaySetu from disclosing the Jitsi room before admission.
Because the application currently uses the public
meet.jit.siservice, an admitted participant could still manually share the disclosed Jitsi room URL. Complete provider-level enforcement would require a self-hosted Jitsi secure-domain or JWT-based deployment.Closes #1628
Type of change
Testing
git diff --checkcompleted without whitespace errors.LiveHearing.test.jsxfor the secured conduct-page navigation flow; 2/2 targeted tests pass locally.Existing repository blockers
The complete frontend production build is currently blocked by pre-existing JSX syntax errors in
VakilFriendPage.jsx, which was not modified in this PR.Backend Maven test execution is currently blocked during production compilation by pre-existing missing NetworkNT JSON Schema classes. No new waiting-room compilation errors were reported before the baseline dependency failure.
Checklist