Skip to content

Comments

Release 1.6.3#93

Merged
bitbacchus merged 20 commits intoproductionfrom
main
Jun 11, 2025
Merged

Release 1.6.3#93
bitbacchus merged 20 commits intoproductionfrom
main

Conversation

@bitbacchus
Copy link
Member

[Release] Merge main into production – 2025-06-11

We have deployed a new version to production! This release merges the latest changes from the main branch, bringing new features, improvements, and bug fixes.

Highlights

  • Deployment & Workflow

    • New workflow for deploying to the test server: .github/workflows/deploy-test.yml
    • Old deployment check workflow removed: .github/workflows/deploy-check.yml
    • Major refactor of deployment.sh for easier log management and improved robustness
  • Backend

    • Backend Dockerfile now uses node:20-slim (was node:20-alpine)
    • Added wkhtmltopdf and related font support to backend Docker image
    • Backend version bumped to 1.0.1
  • Frontend

    • Improved token refresh and handling in TokenActor.ts for more reliable authentication
    • Modernized app root handling and user experience (Root.tsx): better error handling and loading screen
    • Only enable question stats in quiz tab if details are available
    • Frontend version bumped to 1.6.3

For a complete list of changes, see the commit history between main and production.

bitbacchus added 19 commits May 26, 2025 21:45
* update dependencies

* adds loading spinner to dashboard

* Hotfix: Stats details can only be clicked when stats avail.
Updated for Docker deployment
Logrotate function explicitly returns 0 now.
…ization (#89)

* PR version 1.6.2 (#88)

* Hotfix Can only click on stats detaiils when availible (#85)

* update dependencies

* adds loading spinner to dashboard

* Hotfix: Stats details can only be clicked when stats avail.

* Revert "Hotfix Can only click on stats detaiils when availible (#85)" (#86)

This reverts commit 80a01a1.

* Delete .github/workflows/deploy-check.yml

Updated for Docker deployment

Logrotate function explicitly returns 0 now.

* 🔒 Improve token handling in TokenActor: retry on failure and clear previous timeout (#87)

* Update deploy-test.yml

* Version 1.6.2

* 🩹 fix(Root): defer auth cookie check to useEffect after actor initialization

- Move the inline  →  redirect out of the render path
- Perform the login‐redirect in a  that runs once after
- Prevents spurious redirects on re‐renders (e.g. when using React DevTools “pause”)
 🐛 Add ternary operator to conditionally disable question details button
* chore(docker): switch to node:20-slim and install wkhtmltopdf
- solves crash during export
* feat(Root): show spinner while initializing and refactor auth redirect

* :chore: issues with auto-deploy on the test server

* ✨ chore: issues with auto-deploy to testserver

* Update deploy-test.yml

* ✨ chore: issues with auto-deploy to testserver

* version bump
Comment on lines 17 to 38
runs-on: ubuntu-latest
env:
BRANCH_NAME: ${{ github.event.inputs.branch || github.head_ref || github.ref_name }}

steps:
- name: Set up SSH agent
uses: webfactory/ssh-agent@v0.7.0
with:
ssh-private-key: ${{ secrets.TEST_SSH_PRIVATE_KEY }}

- name: Add test server to known_hosts
run: |
mkdir -p ~/.ssh
echo "${{ secrets.TEST_SSH_KNOWN_HOSTS }}" >> ~/.ssh/known_hosts

- name: Deploy ${{ env.BRANCH_NAME }} to test server
run: |
echo "➡️ Starting remote deployment of branch $BRANCH_NAME"
ssh -o StrictHostKeyChecking=no \
${{ secrets.TEST_SERVER_USER }}@${{ secrets.TEST_SERVER_HOST }} \
"export BRANCH_NAME='${BRANCH_NAME}'; bash ~/deploy_recapp_to_test.sh '${BRANCH_NAME}'"
echo "✅ Remote deployment of branch $BRANCH_NAME succeeded"

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI 9 months ago

To fix the issue, we need to add a permissions block to the workflow. This block should specify the least privileges required for the workflow to function correctly. Since the workflow does not interact with repository contents or pull requests, the permissions can be set to contents: read as a minimal starting point. This ensures that the GITHUB_TOKEN has limited access.

The permissions block should be added at the root level of the workflow to apply to all jobs, as there is only one job (deploy) in this workflow.


Suggested changeset 1
.github/workflows/deploy-test.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/deploy-test.yml b/.github/workflows/deploy-test.yml
--- a/.github/workflows/deploy-test.yml
+++ b/.github/workflows/deploy-test.yml
@@ -14,2 +14,5 @@
 
+permissions:
+  contents: read
+
 jobs:
EOF
@@ -14,2 +14,5 @@

permissions:
contents: read

jobs:
Copilot is powered by AI and may make mistakes. Always verify output.
Unable to commit as this autofix suggestion is now outdated
@bitbacchus bitbacchus self-assigned this Jun 11, 2025
@bitbacchus bitbacchus merged commit c81a484 into production Jun 11, 2025
6 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.

1 participant