Skip to content

Create pgweb app #329

Merged
pantherman594 merged 17 commits into
masterfrom
jczerk/db_guis
Mar 2, 2026
Merged

Create pgweb app #329
pantherman594 merged 17 commits into
masterfrom
jczerk/db_guis

Conversation

@pantherman594
Copy link
Copy Markdown
Contributor

Creates a pgweb app for exploring aurora databases in workbench

PHP-124828

@@ -0,0 +1,165 @@
#!/bin/bash
set -o errexit
set -o pipefail
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: set -o nounset

@pantherman594 pantherman594 marked this pull request as ready for review March 2, 2026 13:46
@pantherman594 pantherman594 requested review from a team as code owners March 2, 2026 13:46
@pantherman594 pantherman594 merged commit 2c8a708 into master Mar 2, 2026
3 checks passed
@pantherman594 pantherman594 deleted the jczerk/db_guis branch March 2, 2026 13:46
"postStartCommand": "/workspace/start-bookmark-refresh.sh",
"features": {
"ghcr.io/devcontainers/features/common-utils:2": {
"installZsh": false,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add some comments for why these are disabled?

Comment thread src/pgweb/README.md

## Automatic Database Discovery

The app automatically discovers all Aurora databases in your Workbench workspace and creates pre-configured connection bookmarks with fresh IAM authentication tokens.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: add a link to the original github wiki https://github.com/sosedoff/pgweb/wiki/Server-Connection-Bookmarks for a little more context about this app.

echo "Starting bookmark refresh for pgweb..."

# Create base directory (but not bookmarks subdirectory - that will be a symlink)
mkdir -p /root/.pgweb
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: we set PGWEB_BASE in the other one but not this one. but it doesn't look like we are override WB_EXE and PGWEB_BASE anyway. maybe just keep them as readonly const to be consistent.

pantherman594 added a commit that referenced this pull request Mar 3, 2026
Addresses feedback left on #329

Also downgrades to pgweb user and shows a disabled prompt when attempting to access an aurora database in another region

<img width="1856" height="1681" alt="image" src="https://github.com/user-attachments/assets/41b15d55-d570-45a6-89e0-e2cb3b5dbfea" />
PeterSu92 pushed a commit that referenced this pull request May 21, 2026
* Add pgweb app.

* Add bookmarks for workspace database instances.

* fixup

* fixup

* fixup

* fixup

* fixup

* fixup

* fixup

* fixup

* fixup

* fixup

* fixup

* doc update

* fixup

* Clean up shellcheck disables

* nounset

---------

Co-authored-by: github-actions <github-actions@github.com>
PeterSu92 pushed a commit that referenced this pull request May 21, 2026
Addresses feedback left on #329

Also downgrades to pgweb user and shows a disabled prompt when attempting to access an aurora database in another region

<img width="1856" height="1681" alt="image" src="https://github.com/user-attachments/assets/41b15d55-d570-45a6-89e0-e2cb3b5dbfea" />
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