🐛 Bug Report: query-redis-labs hardcodes Redis port 14714
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 72/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- node.js, redis
- Domain
- backend
Research direction
Start with node/query-redis-labs/src/redis.js and trace how the Redis client gets its connection settings; the issue says its port is fixed at 14714. Check how the function template supplies REDIS_HOST and REDIS_PASSWORD, then decide how to expose the database's port or accept a connection URL. Done when Redis Cloud databases using their assigned ports can connect through the template configuration.
Written by the indexing model from the issue text.
Description
Summary
query-redis-labs can only connect to a Redis Cloud database that happens to use port 14714.
Details
node/query-redis-labs/src/redis.jscreates the client withport: 14714(same at1.2.0andmain).- Redis Cloud assigns each database its own port, and the Console template only asks for
REDIS_HOSTandREDIS_PASSWORD. That config lives inapp/config/templates/function.phpin appwrite/appwrite.
Suggested fix
Read the port from a REDIS_PORT variable and add it to the template's variables in appwrite/appwrite, or accept a full connection URL.
Notes
This turned up while adding templates to the appwrite.io integrations catalog. The Redis page is held back until this is fixed.
cc @adityaoberai
- Dominant language
- HTML
- Stars
- 174
- Forks
- 129
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 11
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from appwrite/templates
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
bug python
Difficulty 3/5 1-2 days Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Dependency DashboardOpen
Difficulty 5/5 Over a week Newbie friendliness 20/100
Maintainers usually reply within 1 day
-
Enhancement: Add TypeScript definitions (IntelliSense) to Node.js starterPossibly taken @Ram3ez claimed this 326 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
Maintainers usually reply within 1 day
-
enhancement
Difficulty 3/5 1-2 days Newbie friendliness 45/100
Maintainers usually reply within 1 day
All issues in appwrite/templates
Similar issues
-
Windows flake: withFileLock throws EPERM when the previous holder's lock file is still being deletedOpen
Difficulty 2/5 1-3 hours Newbie friendliness 79/100
Maintainers usually reply within 1 day
-
🐛 bug
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
LibreChat-AI/LibreChat#17875 ·
Maintainers usually reply within 1 day
-
category:helper tech-debt
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
LanternOps/breeze#11285 ·
Maintainers usually reply within 1 day
-
Use registered media types for HDF5 and NetCDF, and stop claiming media_type must be IANA-registeredOpendocumentation metadata
Difficulty 2/5 1-3 hours Newbie friendliness 87/100
Maintainers usually reply within 1 day
-
idea-scout idea:251
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
jeanibarz/agent-notebook#251 ·
Maintainers usually reply within 1 day