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

composer updates for security #132

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

elizoller
Copy link
Member

GitHub Issue: (link)

What does this Pull Request do?

Composer updates for security

What's new?

Composer packges

How should this be tested?

  • pull in this PR
  • run composer install
  • make sure that Houdini still works (generate image derivatives)

Additional Notes:

supersedes #125 , #126 , #127

Interested parties

@Islandora/8-x-committers

@codecov
Copy link

codecov bot commented Oct 21, 2021

Codecov Report

Merging #132 (d113e95) into 2.x (ee9ea25) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                2.x     #132   +/-   ##
=========================================
  Coverage     77.55%   77.55%           
  Complexity      159      159           
=========================================
  Files             6        6           
  Lines           655      655           
=========================================
  Hits            508      508           
  Misses          147      147           

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 ee9ea25...d113e95. Read the comment docs.

@seth-shaw-unlv
Copy link
Contributor

I attempted to apply this to a running isle-dc install by logging into the houdini container, updating the lock files, and then running composer install but it failed:

The following exception probably indicates you are offline or have misconfigured DNS resolver(s)

                                                                                                                   
  [Composer\Downloader\TransportException]                                                                         
  curl error 6 while downloading https://flex.symfony.com/versions.json: Could not resolve host: flex.symfony.com  

So, presumably I'd need to do this in build-kit?

Someone with a working playbook install want to give this a spin? @alxp or @whikloj?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants