sync libinjection to 1.1.0 about ironbee HOT 6 CLOSED

Hi Nick,

Thanks for the update! I just pull it all in with "git-subtree" to make it easy for us to sync up. At some point we will just make it a dependency, but for now is just easier to include the source as a whole instead of picking choosing. See the libinjection section in DEVELOPMENT.md.

I will sync up with v1.1.0 in our soon-to-be-stable 0.7.x branch.

All I had to do to merge this in was:

git subtree pull --squash --prefix=libs/libinjection libinjection v1.1.0

Looks like I'll have to work around some issue with the missing normalization header.

from ironbee.

I sync'd ironbee/0.7.x with libinjection/v1.1.0 and ironbee/master with libinjection/master and pushed.

Thanks again, Nick!

from ironbee.

Thanks for the tips...

Missing normalization header?
Which one? It should be "complete" just going a checkout.

On 2013/05/04, at 13:39, Brian Rectanus [email protected] wrote:

Hi Nick,

Thanks for the update! I just pull it all in with "git-subtree" to make it easy for us to sync up. At some point we will just make it a dependency, but for now is just easier to include the source as a whole instead of picking choosing. See the libinjection section in DEVELOPMENT.md.

I will sync up with v1.1.0 in our soon-to-be-stable 0.7.x branch.

All I had to do to merge this in was:

git subtree pull --squash --prefix=libs/libinjection libinjection v1.1.0
Looks like I'll have to work around some issue with the missing normalization header.

�$B!=�(B
Reply to this email directly or view it on GitHub.

from ironbee.

sqli_normalize.h

See: e73a9eb

from ironbee.

Oh ok let's talk as that is not needed anymore! It doesn't hurt but you
don't need to make a copy anymore == faster.

Don't need to upcase... If its urlencoded in a query string, then you need
to decode it. But i suspect you did that already. That's it.

Ill be online later to review how to use it.

On Saturday, May 4, 2013, Brian Rectanus wrote:

sqli_normalize.h

See: e73a9ebhttps://github.com/ironbee/ironbee/commit/e73a9ebff82242662f554247c2ff10641556c4ed

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/37#issuecomment-17428575
.

from ironbee.

Oh, I got it all figured out, sorry for any confusion. I got to remove a copy and some other code, so I am happy :)

from ironbee.