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

Already on GitHub? Sign in to your account

Real security fix, and add admin support. #40

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

pahaz commented Jun 1, 2012

No listing dirictories.
Add support django admin.
Add readme =)

pahaz added some commits Jun 1, 2012

@pahaz pahaz reg exp fix! 915fa56
@pahaz pahaz sync
sync from windows
de69067
@pahaz pahaz Security fix and add Admin support
Add Admin interface support.
0a7d3e4
Contributor

pahaz commented Jun 4, 2012

I use this version. And tested it.

Owner

wardi commented Jun 8, 2012

What's with the "sync" commit?

This is cool, but it should be optional

It's also not part of the security fix!

Owner

wardi commented Jun 8, 2012

merged the parts that were obviously security-related.

If I missed something please let me know.

@wardi wardi closed this Jun 8, 2012

Contributor

pahaz commented Jun 9, 2012

sync - this is bug in git windows client

This is cool, but it should be optional
It's also not part of the security fix!

Yes, sorry.

If I missed something please let me know.

I'm sorry, but I do not have the opportunity to watch in the near future.

Contributor

pahaz commented Jun 9, 2012

I found a strange behavior when using Unicode.
Now it does not work if you have time can get better.
But that is another issue.

Need tests =(

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