[PR #2714] [MERGED] Keycloak: Update installation script #3846

Closed
opened 2025-11-20 06:07:00 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/2714
Author: @tremor021
Created: 2/27/2025
Status: Merged
Merged: 2/27/2025
Merged by: @MickLesk

Base: mainHead: keycloak


📝 Commits (5)

📊 Changes

1 file changed (+33 additions, -22 deletions)

View changed files

📝 install/keycloak-install.sh (+33 -22)

📄 Description

✍️ Description

Updated the JDK to v21, as per documentation: https://www.keycloak.org/getting-started/getting-started-zip
Updated script format up to the standard

Link: #

Prerequisites

Before this PR can be reviewed, the following must be completed:

  • [] Self-review performed – Code follows established patterns and conventions.
  • [] Testing performed – Changes have been thoroughly tested and verified.

🛠️ Type of Change

Select all that apply:

  • [] 🆕 New script – A fully functional and tested script or script set.
  • [] 🐞 Bug fix – Resolves an issue without breaking functionality.
  • [] New feature – Adds new, non-breaking functionality.
  • [] 💥 Breaking change – Alters existing functionality in a way that may require updates.

📋 Additional Information (optional)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE/pull/2714 **Author:** [@tremor021](https://github.com/tremor021) **Created:** 2/27/2025 **Status:** ✅ Merged **Merged:** 2/27/2025 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `keycloak` --- ### 📝 Commits (5) - [`06545cb`](https://github.com/community-scripts/ProxmoxVE/commit/06545cb990f19932cf45a95aa38ea4f5db2b030c) Update installation script - [`11e3d88`](https://github.com/community-scripts/ProxmoxVE/commit/11e3d88d2b23059f3c6dfb18efe6190485fa46ef) fixed tar command - [`c86c1c3`](https://github.com/community-scripts/ProxmoxVE/commit/c86c1c3a827af6a9276d2623ca740ef4726c81e5) fix tar again - [`2a8e325`](https://github.com/community-scripts/ProxmoxVE/commit/2a8e32522c39f04ab478cc845cc536a40688e780) fixes - [`7030a4c`](https://github.com/community-scripts/ProxmoxVE/commit/7030a4c3cb15d82b4e91972b669b2c783bfb7957) Update keycloak-install.sh ### 📊 Changes **1 file changed** (+33 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `install/keycloak-install.sh` (+33 -22) </details> ### 📄 Description ## ✍️ Description Updated the JDK to v21, as per documentation: https://www.keycloak.org/getting-started/getting-started-zip Updated script format up to the standard ## 🔗 Related PR / Discussion / Issue Link: # ## ✅ Prerequisites Before this PR can be reviewed, the following must be completed: - [✅] **Self-review performed** – Code follows established patterns and conventions. - [✅] **Testing performed** – Changes have been thoroughly tested and verified. ## 🛠️ Type of Change Select all that apply: - [] 🆕 **New script** – A fully functional and tested script or script set. - [] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [✅] ✨ **New feature** – Adds new, non-breaking functionality. - [] 💥 **Breaking change** – Alters existing functionality in a way that may require updates. ## 📋 Additional Information (optional) <!-- Provide extra context, screenshots, or references if needed. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 06:07:00 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#3846