Conversation
|
@LukasReschke, thanks for your PR! By analyzing the annotation information on this pull request, we identified @MorrisJobke and @rullzer to be potential reviewers |
|
Ref guzzle/guzzle#1393 for the Guzzle error that upstream exists. But since we updated Guzzle it may be already fixed. |
Details |
|
phpseclib/phpseclib#1013 has the phpseclib patches |
|
phpseclib/phpseclib@15c9f99 is the breaking phpseclib commit |
|
PHP 5.4 fails: Details |
|
As #836 (comment) says 😉 – This is caused by an change in phpseclib behaviour. Need to think about how to handle this… |
.htaccess
Outdated
| #### DO NOT CHANGE ANYTHING ABOVE THIS LINE #### | ||
|
|
||
| ErrorDocument 403 /core/templates/403.php | ||
| ErrorDocument 404 /core/templates/404.php |
There was a problem hiding this comment.
Ah... bad autotests.sh touching that file all the time.
|
@rullzer Is this ready for review? |
|
Nope it is an ugly ugly work around. But i have ideas on how to clean up a bit.. Will do soonish |
|
Ok ready for reviews. See https://tools.ietf.org/html/rfc3447#page-38 for the salt length of 0. |
|
LGTM |
* Set salt length to 0 See https://tools.ietf.org/html/rfc3447#page-38 * Fix tests for new phpunit
|
Rebased on top of merged 3rdparty and fixed a merge conflict |
Bumps the PHP version from beta2 to RC1
|
Failing on the updated PHP 7.1 RC1, the errors are strange though. Mhm. 🙈 |
|
Seems like JSON_PRETTY_PRINT behaves differently now… |
In php 7.1 the pretty print json output changed. Thus now we compare arrays.
|
Looks good 👍 |
|
🎉 |
Build should fail. But we can then step by step add fixes and merge when green :)
cc @MorrisJobke