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

Possible security issue with PHP CRUD API #366

Closed
noobiek opened this issue Apr 9, 2018 · 3 comments
Closed

Possible security issue with PHP CRUD API #366

noobiek opened this issue Apr 9, 2018 · 3 comments
Assignees
@mevdschee
Labels
bug

Comments

@noobiek
Copy link
Contributor

noobiek commented Apr 9, 2018 •
edited

Hi Maurits,

I've just sent you an email titled "Possible security issue with PHP CRUD API!"
Please check it ASAP.

Thanks.
Grigoriy

P.S. 2all: I don't want to raise any panic just yet as I might be wrong. I asked Maurits to check my findings first.
@noobiek noobiek changed the title Possible secutiry issue with PHP CRUD API! Possible security issue with PHP CRUD API! Apr 9, 2018
@mevdschee
Copy link
Owner

Thank you. I'm looking into this.

@mevdschee mevdschee changed the title Possible security issue with PHP CRUD API! Possible security issue with PHP CRUD API Apr 9, 2018
@mevdschee mevdschee self-assigned this Apr 9, 2018
@mevdschee mevdschee added the bug label Apr 9, 2018
mevdschee added a commit that referenced this issue Apr 9, 2018
@mevdschee
fix for #366
5b3f545
@mevdschee
Copy link
Owner

mevdschee commented Apr 9, 2018 •
edited

@noobiek can you verify that this fix resolves the issue? Please check version: v1.2.16

@noobiek
Copy link
Contributor Author

noobiek commented Apr 9, 2018

Looks nice!
I'll play with it some more later and let you know if I find anything.

For now, I guess, we can close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mevdschee mevdschee

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mevdschee @noobiek