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.

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

Bump phpseclib/phpseclib from 2.0.13 to 2.0.14 #34284

Merged
merged 1 commit into from
Feb 18, 2019

Conversation

dependabot-preview[bot]
Copy link
Contributor

@dependabot-preview dependabot-preview bot commented Jan 28, 2019

Bumps phpseclib/phpseclib from 2.0.13 to 2.0.14.

Release notes

Sourced from phpseclib/phpseclib's releases.

2.0.14

  • SSH2: ssh-rsa is sometimes incorrectly used instead of rsa-sha2-256 (#1331)
  • SSH2: more strictly adhere to RFC8332 for rsa-sha2-256/512 (#1332)
Changelog

Sourced from phpseclib/phpseclib's changelog.

2.0.14 - 2019-01-27

  • SSH2: ssh-rsa is sometimes incorrectly used instead of rsa-sha2-256 (#1331)
  • SSH2: more strictly adhere to RFC8332 for rsa-sha2-256/512 (#1332)
Commits
  • 8ebfcad Merge branch '1.0' into 2.0
  • 7432a69 1.0.14 release
  • 5c5a8a5 Merge branch '1.0' into 2.0
  • 0f87a0e SSH2: CS adjustment
  • 004a71c Merge branch '1.0' into 2.0
  • e5ff894 SSH2: add missing constant
  • f8ff1aa Merge branch '1.0' into 2.0
  • b0d63fb SSH2: ssh-rsa is sometimes incorrectly used instead of rsa-sha2-256
  • 31fbdb9 Merge branch '1.0' into 2.0
  • 055d609 SSH2: more strictly adhere to RFC8332 for rsa-sha2-256/512
  • See full diff in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot ignore this [patch|minor|major] version will close this PR and stop Dependabot creating any more for this minor/major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Pull request limits (per update run and/or open at any time)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

Finally, you can contact us by mentioning @dependabot.

@dependabot-preview dependabot-preview bot added dependencies php Pull requests that update Php code labels Jan 28, 2019
@codecov
Copy link

codecov bot commented Jan 28, 2019

Codecov Report

Merging #34284 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #34284   +/-   ##
=========================================
  Coverage     64.76%   64.76%           
  Complexity    18368    18368           
=========================================
  Files          1199     1199           
  Lines         69550    69550           
  Branches       1281     1281           
=========================================
  Hits          45044    45044           
  Misses        24133    24133           
  Partials        373      373
Flag Coverage Δ Complexity Δ
#javascript 53.09% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 66.11% <ø> (ø) 18368 <ø> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7048a4...a745d34. Read the comment docs.

@codecov
Copy link

codecov bot commented Jan 28, 2019

Codecov Report

Merging #34284 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #34284   +/-   ##
=========================================
  Coverage     64.78%   64.78%           
  Complexity    18372    18372           
=========================================
  Files          1199     1199           
  Lines         69565    69565           
  Branches       1281     1281           
=========================================
  Hits          45065    45065           
  Misses        24127    24127           
  Partials        373      373
Flag Coverage Δ Complexity Δ
#javascript 53.09% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 66.13% <ø> (ø) 18372 <ø> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46255e2...862aa4b. Read the comment docs.

@dependabot-preview dependabot-preview bot force-pushed the dependabot/composer/phpseclib/phpseclib-2.0.14 branch 2 times, most recently from 73c23f1 to fa4ddd4 Compare February 5, 2019 14:40
@phil-davis
Copy link
Contributor

Note: see #34265 (comment) by @PVince81

@phil-davis
Copy link
Contributor

@dependabot recreate

@dependabot-preview dependabot-preview bot force-pushed the dependabot/composer/phpseclib/phpseclib-2.0.14 branch from fa4ddd4 to 862aa4b Compare February 8, 2019 05:46
@phil-davis phil-davis requested a review from PVince81 February 8, 2019 07:43
@PVince81
Copy link
Contributor

PVince81 commented Feb 8, 2019

@phil-davis please confirm that SFTP tests ran and passed

@phil-davis phil-davis self-assigned this Feb 14, 2019
@phil-davis
Copy link
Contributor

@PVince81 I got files_external_ftp unit tests passing again. See owncloud/files_external_ftp#33 and my comments on that.
That runs some SFTP stuff.
And core tests that use tests/lib/Files/External/Service/StoragesServiceTest class pass for me locally. That class has SFTP stuff in it.

What else should be run?

@PVince81
Copy link
Contributor

@phil-davis actually it's https://github.com/owncloud/core/blob/master/apps/files_external/tests/Storage/SftpTest.php

I wasn't sure whether you were familiar how we ran the tests. Let's have a chat, I need to search for old info.

@PVince81
Copy link
Contributor

seems not all external tests got ported to Drone.

here we have SMB: https://github.com/owncloud/core/blob/stable10/tests/drone/configs/config.files_external.smb-windows.php#L2

you can run SFTP locally with: make test-external TEST_EXTERNAL_ENV=sftp-atmoz

the latter refers to the files in https://github.com/owncloud/core/tree/master/apps/files_external/tests/env where you remove "start-" from "start-sftp-atmoz" so it becomes "sftp-atmoz".

I think we should bring back those tests in some form, maybe run them daily only on stable10.

Thoughts ? @phil-davis @individual-it

@PVince81
Copy link
Contributor

raised #34522 for automation.

for now we should run these tests locally

@PVince81 PVince81 assigned PVince81 and unassigned phil-davis Feb 18, 2019
@PVince81
Copy link
Contributor

% make test-external TEST_EXTERNAL_ENV=sftp-atmoz
PHPUNIT="/srv/www/htdocs/owncloud/lib/composer/phpunit/phpunit/phpunit" build/autotest-external.sh sqlite sftp-atmoz 
Using database oc_autotest
Setup environment for sqlite testing ...
cp: cannot stat '/srv/www/htdocs/owncloud/tests/autoconfig-sqlite.php': No such file or directory
Installing ....
creating sqlite db
ownCloud was successfully installed
Testing with sqlite ...
Run only sftp-atmoz ...
files_external enabled
No coverage
PHPUnit 5.7.27 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.2.14
Configuration: /srv/www/htdocs/owncloud/tests/phpunit-autotest-external.xml

...............................................................   63 / 63 (100%)

Time: 297 ms, Memory: 18.00MB

OK (63 tests, 128 assertions)
start: start-sftp-atmoz.sh
name: sftp-atmoz
Docker executable found - setup docker
Fetch recent atmoz/sftp docker image
Using default tag: latest
latest: Pulling from atmoz/sftp
Digest: sha256:fb2645743918a40991475c6a079aab180218e233b9f968e52b7693294a42626a
Status: Image is up to date for atmoz/sftp:latest
sftp container: 2d89604c96ab02ab15a6c7261ce841662a4bce91f6980c57daee1fa1a8d755a9
Waiting for sftp initialization.....

@PVince81 PVince81 merged commit 13375f1 into master Feb 18, 2019
@delete-merged-branch delete-merged-branch bot deleted the dependabot/composer/phpseclib/phpseclib-2.0.14 branch February 18, 2019 07:02
@lock lock bot locked as resolved and limited conversation to collaborators Feb 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3 - To Review dependencies php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants