suhosin7 icon indicating copy to clipboard operation
suhosin7 copied to clipboard

make build possible on 7.2 and up...

Open mcarbonneaux opened this issue 6 years ago • 3 comments

for now suhosin7 not build on php 7.2, this patch make possible to build on 7.2 and up...

mcarbonneaux avatar Aug 22 '18 00:08 mcarbonneaux

i reviewed and tested, seems build and works with php7.2, travis test said fails two checks:

FAIL session id too long [tests/session/max_id_length_toolong.phpt] FAIL session user handler recursive crash - issue suhosin#60 [tests/session/session_recursive_crash2.phpt]

seems are very important the test failed but in certaint environment does not fail!

mckaygerhard avatar Aug 30 '18 14:08 mckaygerhard

i've seen this test, i've not investigated why session id has changed in size... possible 7.2 session hash size as differente size for now... when i've some time i while investigade in the code why.

mcarbonneaux avatar Aug 31 '18 19:08 mcarbonneaux

maybe will be necessary ask to php developers directly

mckaygerhard avatar Aug 31 '18 19:08 mckaygerhard