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

[1.2.4] [FIX] ACL allows access to forbidden resources #1305

Merged
merged 5 commits into from Sep 28, 2013
Merged

[1.2.4] [FIX] ACL allows access to forbidden resources #1305

merged 5 commits into from Sep 28, 2013

Conversation

ghost
Copy link

@ghost ghost commented Sep 27, 2013

See #1303

@sjinks
Regression test
37e6ae0 
(cherry picked from commit ee13afee1ef9d8dcd473a3ff5c923e481a5b842e)
@sjinks
Regression test for #65
224452b 
(cherry picked from commit 14b82c5ad05d6c65bd9306e5c5ff5b9c0286ed4f)
@sjinks
Fix #1303
184a716 
(cherry picked from commit e3101a6a95da39ce77b97c76d32c5db90a4f53b0)

Conflicts:
	ext/acl/adapter/memory.c
@sjinks
issue-1261.phpt requires openssl
5166a0b 
(cherry picked from commit 7dce086)
@sjinks
Regenerated build/ [ci skip]
43ff648
phalcon pushed a commit that referenced this pull request Sep 28, 2013
Merge pull request #1305 from sjinks/1.2.4
e22c0de 
[1.2.4] [FIX] ACL allows access to forbidden resources
@phalcon phalcon merged commit e22c0de into phalcon:1.2.4 Sep 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@phalcon @sjinks