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

v3.0.0 #40

Merged
merged 174 commits into from
Jan 29, 2024
Merged

v3.0.0 #40

merged 174 commits into from
Jan 29, 2024

Conversation

prlanzarin
Copy link
Member

v3.0.0

  • feat: new xAPI output module with support for multitenancy
  • feat(xapi): add suport for meta_xapi-create-end-actor-name
  • feat(webhooks): implement includeEvents/excludeEvents
  • feat(events): add support for poll events
  • feat(events): add support for raise-hand events
  • feat(events): add support for emoji events
  • feat(events): add user info to screenshare events
  • feat(events): add support for audio muted/unmuted events
  • feat: support internal_meeting_id != record_id on rap events
  • feat: add Prometheus instrumentation
  • feat: add JSDoc annotations to most of the codebase
  • feat: log to file
  • feat: add support for multiple checksum algorithms (SHA1,...,SHA512)
  • feat(test): add support for modular test suites
  • feat(test): add xAPI test suite
  • feat: pipelines with GitHub Actions
  • !refactor: application rewritten to use a modular input/processing/ouput system
  • !refactor: modernize codebase (ES6 imports, Node.js >= 18 etc.)
  • !refactor(webhooks): the webhooks functionality was rewritten into an output module
  • !refactor(webhooks): hook IDs are now UUIDs instead of integers
  • !refactor: new logging system (using Pino)
  • !refactor: migrate node-redis from v3 to v4
  • !refactor: new queue system (using Bullmq)
  • refactor(test): remove nock as a dependency
  • refactor(webhooks): replace request with node-fetch
  • refactor: replace sha1 dependency with native code
  • refactor: remove unused events
    • rap-published, rap-unpublished, rap-deleted
  • !fix(webhooks): remove general getRaw configuration
  • fix(events): user-left events are now emitted for trailing users on meeting-ended events
  • fix(test): restore remaining out/webhooks tests
  • fix: add Redis disconnection handling
  • build: add docker-compose and updated Dockerfile examples
  • build: set .nvmrc to lts/iron (Node.js 20)

prlanzarin and others added 30 commits September 26, 2023 11:23
e.g.: let an user define input and output modules in the same module config/pkg
Porting 445e30c into v3
  - add parent-id to meeting-created
  - add userdata to user-joined
  - add emoji to user-emoji-changed
  - add id and name to chat-group-message-sent
  - add recorded and duration to rap-archive-ended
  - fix start-time, end-time and raw-size on rap-publish-ended
  - Replaces request (unmaintained) with node-fetch (maintained,
    slimmer and should provide us with a clean transition to Node.js
    fetch when that is stable)
  - Restore the domain field in hooks
  - Restore the bearer auth option
  - Add nodemon and a start-dev script to enable auto-reload of the
    application on code changes
mprytoluk and others added 28 commits November 28, 2023 10:02
(xAPI) - ensure the correct lrs_endpoint is used
xAPI: add support for meta_xapi-create-end-actor-name
includeEvents/excludeEvents were feature flags present in the previous
version (v2). Those flags provided filter in/out lists for events
generated by the webhooks module and weren't ported to the rewritten
bbb-webhooks.
There's an unused global getRaw configuration in the webhooks module
that conflicted with the hook-specific getRaw parameter. It superseded
the specific configuration, which caused inconsistencies depending on
base configuration being used.

Remove the general getRaw configuration since it's unused and doesn't
really make sense - and fix the inconsistency as a side effect.
chore: remove unneeded bash installation from container

feature: add pipeline to run the tests

feature: add hadolint to build pipeline

fix: copy configs for testing

chore: split jobs

chore: remove build cache

chore: update build-push-action to v5

fix: unauthorized to push image

fix: syntax

feature: trivy pipeline

fix: provide write access to trivy to upload the vulnerability report

chore: add cache to build

chore: update permissions and severity to trivy report

fix: adjust cache

chore: add trivy scanner to docker image as well

fix: set trivy credentials

fix: trivy format

chore: activate cache

chore: try to use github action cache for the image

refactor: .dockerignore

refactor: updates to build cache, image labels and tags, and conditions when the workflows run

fix: make workflows reusable

fix: yaml syntax

fix: step name key

chore: only build and push after hadolint and tests

fix: add permission to write comments on issues/pr

fix: adjust permissions

chore: do not run workflows on release, only on tag

fix: do not try to add comment to pr if not pr

chore: add docs for the workflows

refactor: change action to add pr comment which doesn't duplicate the same content

Revert "refactor: change action to add pr comment which doesn't duplicate the same content"

This reverts commit 4042453.

chore: update comment message
chore: update github-slug-action to latest version
fix: adjust action triggers for pr
Copy link

Updated Docker image pushed to docker.io/bigbluebutton/bbb-webhooks:pr-40

@prlanzarin prlanzarin merged commit 1cb9cae into main Jan 29, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants