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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Mongo Plugin: don't lowercase params in URI #32145

Closed
1 task done
sumitsum opened this issue Mar 28, 2024 · 0 comments · Fixed by #32375
Closed
1 task done

[Bug]: Mongo Plugin: don't lowercase params in URI #32145

sumitsum opened this issue Mar 28, 2024 · 0 comments · Fixed by #32375
Assignees
@NilanshBansal
Labels
Bug Something isn't working Integrations Product Issues related to a specific integration Low effort Takes 1-2 days Low An issue that is neither critical nor breaks a user flow Mongo Issues related to Mongo DB plugin Needs Triaging Needs attention from maintainers to triage Production QA Pod Issues under the QA Pod QA Needs QA attention Query & JS Pod Issues related to the query & JS Pod

Comments

@sumitsum
Copy link
Contributor

sumitsum commented Mar 28, 2024 •
edited by NilanshBansal
Loading

Is there an existing issue for this?

  • I have searched the existing issues

Description

MongoPlugin automatically lowercases the URI added by the user.

shot-2024-03-27-13-45-49.mp4

Slack Thread

Steps To Reproduce

Slack Thread

Public Sample App

No response

Environment

Production

Severity

Low (Inconsistent UI/UX)

Version

cloud, self hosted
@sumitsum sumitsum added Bug Something isn't working Needs Triaging Needs attention from maintainers to triage Mongo Issues related to Mongo DB plugin labels Mar 28, 2024
@github-actions github-actions bot added the Integrations Product Issues related to a specific integration label Mar 28, 2024
@Nikhil-Nandagopal Nikhil-Nandagopal added Production Low An issue that is neither critical nor breaks a user flow labels Mar 28, 2024
@github-actions github-actions bot added the Query & JS Pod Issues related to the query & JS Pod label Mar 28, 2024
@rohan-arthur rohan-arthur added the Low effort Takes 1-2 days label Apr 1, 2024
@NilanshBansal NilanshBansal self-assigned this Apr 2, 2024
@NilanshBansal NilanshBansal mentioned this issue Apr 3, 2024
Merged
NilanshBansal added a commit that referenced this issue Apr 8, 2024
@NilanshBansal
fix: mongo uri lowercasing removed (#32375)
b85cf08 
## Description
> This PR prevents the lowercasing of the URI for the plugins. The URI
is now preserved in the same form that the user has added.
Fixes #32145

## Automation

/ok-to-test tags="@tag.Datasource"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!IMPORTANT]  
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/8592797453>
> Commit: `f6dcdc85a4aafdd3329bf7b9de3fa04e583ec340`
> Cypress dashboard url: <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=8592797453&attempt=1"
target="_blank">Click here!</a>
> All cypress tests have passed 🎉🎉🎉

<!-- end of auto-generated comment: Cypress test results  -->







<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Bug Fixes**
- Fixed an issue with option processing by retaining the original case
of keys, ensuring accurate data handling.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@appsmith-bot appsmith-bot added the QA Needs QA attention label Apr 8, 2024
@github-actions github-actions bot added the QA Pod Issues under the QA Pod label Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NilanshBansal NilanshBansal

Labels
Bug Something isn't working Integrations Product Issues related to a specific integration Low effort Takes 1-2 days Low An issue that is neither critical nor breaks a user flow Mongo Issues related to Mongo DB plugin Needs Triaging Needs attention from maintainers to triage Production QA Pod Issues under the QA Pod QA Needs QA attention Query & JS Pod Issues related to the query & JS Pod
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: mongo uri lowercasing removed appsmithorg/appsmith
5 participants
@sumitsum @Nikhil-Nandagopal @NilanshBansal @appsmith-bot @rohan-arthur