Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] develop from Onlineberatung:develop #19

Open
wants to merge 76 commits into
base: develop
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
76 commits
Select commit Hold shift + click to select a range
0f59221
chore: add persistence layer to Videoservice, add spotless
tkuzynow Apr 25, 2023
67f7772
chore: add persistence layer to Videoservice, add spotless
tkuzynow Apr 25, 2023
eab9405
chore: include session id
tkuzynow Apr 26, 2023
2ebe83d
chore: extend APIs for groupchat video start/stop
tkuzynow Apr 27, 2023
af2fe21
chore: validate consultant can join the groupchat
tkuzynow Apr 28, 2023
871c7f9
chore: validate consultant can join the groupchat
tkuzynow Apr 28, 2023
0fa5a49
feat: send message on video call stop
tkuzynow May 8, 2023
f6515c2
fix: fix DB changelog structure
tkuzynow May 9, 2023
5cb6c43
fix: fix DB changelog structure
tkuzynow May 9, 2023
73ae06f
feat: join videocall functionality
tkuzynow May 9, 2023
4f62d3a
feat: join videocall functionality
tkuzynow May 9, 2023
5196cfc
fix: fix directory structure
tkuzynow May 10, 2023
2c336c1
fix: video room creation
tkuzynow May 10, 2023
3d18640
fix: init tables script
tkuzynow May 10, 2023
3cca712
fix: init tables script
tkuzynow May 10, 2023
1edadc9
fix: init tables script
tkuzynow May 10, 2023
ce7ad11
Merge branch 'develop' into OB-1807-add-persistence-layer-to-videoser…
tkuzynow May 12, 2023
fe06d74
fix: send proper event types for videostop and start
tkuzynow May 12, 2023
f610dca
Merge remote-tracking branch 'origin/OB-1807-add-persistence-layer-to…
tkuzynow May 12, 2023
6981f1d
fix: enable jitsi technical to call videocall stop, to be called from…
tkuzynow May 22, 2023
5f85127
fix: separate endpoint for stop video call event handling
tkuzynow May 22, 2023
a6624bd
fix: separate endpoint for stop video call event handling
tkuzynow May 22, 2023
653bc54
fix: separate endpoint for stop video call event handling
tkuzynow May 22, 2023
f647094
fix: implement stateless api client call
tkuzynow May 24, 2023
17819e5
Merge branch 'develop' into OB-5059-fix-videocall-issue
tkuzynow May 24, 2023
5b2a110
Merge pull request #70 from Onlineberatung/OB-5059-fix-videocall-issue
tkuzynow May 24, 2023
e8e679d
fix: web security config for jitsi call
tkuzynow May 26, 2023
0a8eeea
fix: web security config for jitsi call
tkuzynow May 26, 2023
92d0390
fix: close room for one 2 one calls on handleCloseEvent
tkuzynow May 26, 2023
699a5ae
fix: close room for one 2 one calls on handleCloseEvent
tkuzynow May 26, 2023
efcd343
fix: send message containing guest video rooms, send call duration
tkuzynow May 29, 2023
5413625
fix: send initiator rc user id and display name as non-null fields
tkuzynow May 29, 2023
753b892
fix: send event type
tkuzynow May 29, 2023
06152f1
Merge pull request #69 from Onlineberatung/OB-1807-add-persistence-la…
tkuzynow May 30, 2023
2efc622
fix: include rocketchat room id to persistence
tkuzynow May 30, 2023
58b765f
fix: include rocketchat room id to persistence
tkuzynow May 30, 2023
56f438a
fix: call liveservice with userIds from MariaDB, not RocketChat for g…
tkuzynow May 31, 2023
462718a
fix: do not notify initiating user about the call
tkuzynow May 31, 2023
4c413ed
fix: do not notify initiating user about the call
tkuzynow May 31, 2023
d6f4744
feat: add proper security configs for groupchat join
tkuzynow Jun 5, 2023
a9458f8
feat: add proper security configs for reject, fix return status code …
tkuzynow Jun 5, 2023
cd566e0
feat: add proper security configs for reject, fix return status code …
tkuzynow Jun 5, 2023
0a73ff5
Merge pull request #72 from Onlineberatung/OB-1807-video-calls-for-gr…
tkuzynow Jun 5, 2023
7c5b53e
feat: drop not needed column
tkuzynow Jun 7, 2023
7b93890
Merge branch 'develop' into OB-1807-video-calls-for-groupchats
tkuzynow Jun 7, 2023
5b71677
Merge pull request #73 from Onlineberatung/OB-1807-video-calls-for-gr…
tkuzynow Jun 7, 2023
96c80fb
fix: filter stop event
tkuzynow Aug 24, 2023
2dd9f1a
fix: filter stop event
tkuzynow Aug 24, 2023
4b353ca
fix: filter stop event
tkuzynow Aug 24, 2023
3fcdb6a
fix: filter stop event
tkuzynow Aug 27, 2023
105af6c
Merge pull request #76 from Onlineberatung/OB-5739-adjust-videocall-s…
tkuzynow Aug 28, 2023
1cfa273
feat: include adviceSeekerId in start video call event
tkuzynow Aug 29, 2023
7c8a057
fix: typo
tkuzynow Aug 30, 2023
958aeee
Merge pull request #77 from Onlineberatung/OB-5247-extended-data-export
tkuzynow Aug 31, 2023
a9f3983
fix: pass tenantId for startVideoCall event
tkuzynow Sep 14, 2023
ce44911
fix: pass tenantId for startVideoCall event
tkuzynow Sep 15, 2023
5b60cd2
fix: pass tenantId for startVideoCall event
tkuzynow Sep 15, 2023
453d9c9
Merge pull request #78 from Onlineberatung/OB-5247-extended-data-export
tkuzynow Sep 18, 2023
b0bc424
feat: upgrade to java 17 partial commit
tkuzynow Mar 29, 2024
012ea80
feat: upgrade to java 17
tkuzynow Apr 2, 2024
deefa68
feat: upgrade to java 17- changes in workflow
tkuzynow Apr 2, 2024
4804fc1
feat: upgrade to java 17-fix spotless
tkuzynow Apr 2, 2024
f3300f1
feat: upgrade liquibase version
tkuzynow Apr 2, 2024
badc79e
feat: get rid of KeycloakAuthenticationToken, due to migration to new…
tkuzynow Apr 3, 2024
64ec046
fix: enable request logging to simplify debugging
tkuzynow Apr 3, 2024
7c94cc5
fix: remove session id param as required, because it's not required i…
tkuzynow Apr 3, 2024
7ed222b
fix: classCastException for RedirectStategy in messageService
tkuzynow Apr 3, 2024
5285392
fix: code review comments
tkuzynow Apr 5, 2024
adbe2be
fix: code review comments
tkuzynow Apr 5, 2024
12aa0fa
fix: feature branch workflow java version
tkuzynow Apr 5, 2024
c291a00
Merge pull request #80 from Onlineberatung/TSYSTEMS-140-upgrade-video…
tkuzynow Apr 5, 2024
8382134
fix: technical resolver change to switch to new authentication objects
tkuzynow Apr 15, 2024
ecf2059
fix: technical resolver change to switch to new authentication objects
tkuzynow Apr 15, 2024
e866448
Merge pull request #81 from Onlineberatung/TSYSTEMS-140-upgrade-video…
tkuzynow Apr 16, 2024
87c1b0f
Update dockerImage.yml
tkuzynow Sep 27, 2024
5496834
Merge pull request #82 from Onlineberatung/tkuzynow-patch-1
tkuzynow Sep 27, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions .github/workflows/dockerImage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
- name: Setup JVM
uses: actions/setup-java@v1
with:
java-version: 11.0.10
java-version: 17.0.7
java-package: jdk
architecture: x64

Expand All @@ -43,7 +43,7 @@ jobs:
run: mvn -B -Pprod clean package -DskipTests
- name: Maven Verify
run: mvn -B -Pprod clean verify
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v4
with:
name: targetfiles
path: target/*.jar
Expand All @@ -64,7 +64,7 @@ jobs:
push_to_registry:
strategy:
matrix:
registry: ["docker.pkg.github.com", "ghcr.io"]
registry: ["ghcr.io"]
needs: [test]
name: Push Docker image to GitHub Packages
runs-on: ubuntu-latest
Expand All @@ -74,7 +74,7 @@ jobs:
steps:
- uses: actions/checkout@v2
- name: Download buildfiles artifact
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: targetfiles
- name: Get current time
Expand Down Expand Up @@ -182,4 +182,4 @@ jobs:
value: ${{ env.DOCKER_IMAGE_TAG }}
custom-actions: |
- text: View CI
url: "${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}"
url: "${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}"
2 changes: 1 addition & 1 deletion .github/workflows/feature-branch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
- name: Setup JVM
uses: actions/setup-java@v1
with:
java-version: 11.0.10
java-version: 17.0.7
java-package: jdk
architecture: x64

Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,4 @@
/.nb-gradle/
wrapper
log
/src/main/resources/application-local.properties
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM adoptopenjdk/openjdk11
FROM openjdk:17-oracle
VOLUME ["/tmp","/log"]
EXPOSE 8080
ARG JAR_FILE
Expand Down
109 changes: 105 additions & 4 deletions api/videoservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ paths:
content:
'application/json':
schema:
$ref: '#/components/schemas/CreateVideoCallResponseDTO'
$ref: '#/components/schemas/VideoCallResponseDTO'
400:
description: BAD REQUEST - invalid/incomplete request or body object
401:
Expand All @@ -44,6 +44,39 @@ paths:
description: INTERNAL SERVER ERROR - server encountered unexpected condition
security:
- Bearer: [ ]
/videocalls/join/{roomId}:
post:
tags:
- video-controller
summary: 'Join an existing new video call room Id [Authorization: Role: consultant]'
operationId: joinGroupVideoCall
parameters:
- name: roomId
in: path
required: true
schema:
type: string
format: uuid
responses:
200:
description: JOINED - joined successfully to a video call room
content:
'application/json':
schema:
$ref: '#/components/schemas/VideoCallResponseDTO'
400:
description: BAD REQUEST - invalid/incomplete request or body object
401:
description: UNAUTHORIZED - no/invalid Keycloak token
403:
description: 'FORBIDDEN - no/invalid role/authorization, CSRF token or consultant without
permission for given session'
404:
description: NOT FOUND - Room not found
500:
description: INTERNAL SERVER ERROR - server encountered unexpected condition
security:
- Bearer: [ ]

/videocalls/stop/{roomId}:
post:
Expand Down Expand Up @@ -73,6 +106,33 @@ paths:
500:
description: INTERNAL SERVER ERROR - server encountered unexpected condition

/videocalls/event/stop/{roomId}:
post:
tags:
- video-controller
summary: 'Stops a video call [Auth: jitsi-technical]'
description: 'Handles video call stop event fired by Jitsi (Machine2Machine).'
operationId: handleVideoCallStoppedEvent
parameters:
- name: roomId
in: path
required: true
schema:
type: string
responses:
204:
description: NO CONTENT
400:
description: BAD REQUEST - invalid/incomplete request or body object
401:
description: UNAUTHORIZED - no/invalid Keycloak token
403:
description: FORBIDDEN - access denied
404:
description: NOT FOUND - room ID unknown
500:
description: INTERNAL SERVER ERROR - server encountered unexpected condition

/videocalls/reject:
post:
tags:
Expand Down Expand Up @@ -129,20 +189,47 @@ paths:
description: NOT FOUND - rcGroupId unknown
500:
description: INTERNAL SERVER ERROR - server encountered unexpected condition
/videocalls/room/{jitsiRoomId}:
post:
tags:
- video-controller
summary: 'Creates a new room [Authorization: Role: jitsi-technical]'
operationId: createRoom
parameters:
- name: jitsiRoomId
in: path
required: true
schema:
type: long
responses:
200:
description: OK - successful operation
content:
application/json:
schema:
$ref: '#/components/schemas/VideoRoomResponseDTO'
400:
description: BAD REQUEST - invalid/incomplete request or body object
401:
description: UNAUTHORIZED - no/invalid Keycloak token
500:
description: INTERNAL SERVER ERROR - server encountered unexpected condition


components:
schemas:
CreateVideoCallDTO:
type: object
required:
- sessionId
properties:
sessionId:
type: integer
format: int64
groupChatId:
type: integer
format: int64
initiatorDisplayName:
type: string
CreateVideoCallResponseDTO:
VideoCallResponseDTO:
type: object
required:
- moderatorVideoCallUrl
Expand Down Expand Up @@ -178,6 +265,20 @@ components:
jwt:
type: string

VideoRoomResponseDTO:
type: object
properties:
jitsiRoomId:
type: long
example: 12132
videoLink:
type: string
example: 12132
createDate:
type: string
updateDate:
type: string

securitySchemes:
Bearer:
type: apiKey
Expand Down
24 changes: 13 additions & 11 deletions google_checks_light.xml
Original file line number Diff line number Diff line change
Expand Up @@ -175,11 +175,13 @@
<message key="name.invalidPattern"
value="Type name ''{0}'' must match pattern ''{1}''."/>
</module>
<!--
<module name="MemberName">
<property name="format" value="^[a-z][a-z0-9][a-zA-Z0-9]*$"/>
<message key="name.invalidPattern"
value="Member name ''{0}'' must match pattern ''{1}''."/>
</module>
-->
<module name="ParameterName">
<property name="format" value="^[a-z]([a-z0-9][a-zA-Z0-9]*)?$"/>
<message key="name.invalidPattern"
Expand Down Expand Up @@ -226,14 +228,14 @@
<message key="ws.notPreceded"
value="GenericWhitespace ''{0}'' is not preceded with whitespace."/>
</module>
<module name="Indentation">
<property name="basicOffset" value="2"/>
<property name="braceAdjustment" value="0"/>
<property name="caseIndent" value="2"/>
<property name="throwsIndent" value="4"/>
<property name="lineWrappingIndentation" value="4"/>
<property name="arrayInitIndent" value="2"/>
</module>
<!-- <module name="Indentation">-->
<!-- <property name="basicOffset" value="2"/>-->
<!-- <property name="braceAdjustment" value="0"/>-->
<!-- <property name="caseIndent" value="2"/>-->
<!-- <property name="throwsIndent" value="4"/>-->
<!-- <property name="lineWrappingIndentation" value="4"/>-->
<!-- <property name="arrayInitIndent" value="2"/>-->
<!-- </module>-->
<module name="AbbreviationAsWordInName">
<property name="ignoreFinal" value="false"/>
<property name="allowedAbbreviationLength" value="1"/>
Expand Down Expand Up @@ -283,7 +285,6 @@
<property name="tokens" value="VARIABLE_DEF"/>
<property name="allowSamelineMultipleAnnotations" value="true"/>
</module>

<!--
<module name="NonEmptyAtclauseDescription"/>
<module name="InvalidJavadocPosition"/>
Expand All @@ -299,18 +300,18 @@
value="CLASS_DEF, INTERFACE_DEF, ENUM_DEF, METHOD_DEF, CTOR_DEF, VARIABLE_DEF"/>
</module>
-->

<module name="JavadocMethod">
<property name="scope" value="public"/>
<property name="allowMissingParamTags" value="true"/>
<property name="allowMissingReturnTag" value="true"/>
<property name="allowedAnnotations" value="Override, Test"/>
<property name="tokens" value="METHOD_DEF, CTOR_DEF, ANNOTATION_FIELD_DEF"/>
</module>
<!--
<module name="MissingJavadocMethod">
<property name="scope" value="public"/>
<property name="minLineCount" value="2"/>
<property name="allowedAnnotations" value="Override, Test, Before"/>
<property name="allowedAnnotations" value="Override, Test"/>
<property name="tokens" value="METHOD_DEF, CTOR_DEF, ANNOTATION_FIELD_DEF"/>
</module>
<module name="MethodName">
Expand All @@ -321,6 +322,7 @@
<module name="SingleLineJavadoc">
<property name="ignoreInlineTags" value="false"/>
</module>
-->
<module name="EmptyCatchBlock">
<property name="exceptionVariableName" value="expected"/>
</module>
Expand Down
Loading
Loading