Skip to content

Upgrading package from 4.12.0 to 4.13.0 #791

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

Closed
sawilde opened this issue Dec 6, 2023 · 1 comment · Fixed by #792
Closed

Upgrading package from 4.12.0 to 4.13.0 #791

sawilde opened this issue Dec 6, 2023 · 1 comment · Fixed by #792

Comments

@sawilde
Copy link

sawilde commented Dec 6, 2023

How do you use Sentry?

Sentry SaaS (sentry.io)

SDK version

4.13.0

Steps to reproduce

symfony/symfony (4.4.51) - we are planning on upgrading but it is not a tomorrow thing
sentry/sentry-symfony (4.12.0)

upgrade to sentry/sentry-symfony (4.13.0)

Package operations: 0 installs, 2 updates, 0 removals
  - Downloading sentry/sentry-symfony (4.13.0)
  - Upgrading sentry/sdk (3.5.0 => 3.6.0)
  - Upgrading sentry/sentry-symfony (4.12.0 => 4.13.0): Extracting archive

...

In DecoratorServicePass.php line 83:
                                                                                                                                      
  The service "Sentry\SentryBundle\Tracing\HttpClient\TraceableHttpClient" has a dependency on a non-existent service "http_client".  

Not something I would've expected for a minor package update

Expected result

Upgrade works without any code change or the notes within the upgrade contain want needs to be done to take the new version

Actual result

In DecoratorServicePass.php line 83:
                                                                                                                                      
  The service "Sentry\SentryBundle\Tracing\HttpClient\TraceableHttpClient" has a dependency on a non-existent service "http_client".  
@cleptric
Copy link
Member

cleptric commented Dec 6, 2023

Fixed in 4.13.1

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

Successfully merging a pull request may close this issue.

4 participants