Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: select security bug #879

Merged
merged 1 commit into from
Feb 27, 2024
Merged

fix: select security bug #879

merged 1 commit into from
Feb 27, 2024

Conversation

awaismehmood88
Copy link
Contributor

When query select pass column without primary key in response all columns including relations columns shows those not including in config.select.
Issue is fixed will small check if query.select doesn't includes primary key user config.select as columns.

Copy link

codecov bot commented Feb 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.85%. Comparing base (c052d15) to head (32a9c9b).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #879      +/-   ##
==========================================
+ Coverage   97.69%   97.85%   +0.16%     
==========================================
  Files          16       16              
  Lines         650      652       +2     
  Branches      181      182       +1     
==========================================
+ Hits          635      638       +3     
+ Misses         15       14       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ppetzold ppetzold merged commit 6f47f0c into ppetzold:master Feb 27, 2024
5 checks passed
@ppetzold
Copy link
Owner

🎉 This PR is included in version 8.6.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@awaismehmood88 awaismehmood88 deleted the fix-select branch February 27, 2024 10:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants