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

Remove unused em-http-request #4069

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Remove unused em-http-request #4069

merged 1 commit into from
Nov 11, 2024

Conversation

sethboyles
Copy link
Member

@sethboyles sethboyles commented Nov 4, 2024

  • this gem is blocking us from upgrading to Ruby 3.3, but it seems we don't actually use it

#3559

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

* this gem is blocking us from upgrading to Ruby 3.3, but it seems we don't actually use it
@sethboyles sethboyles requested review from a team November 7, 2024 18:02
end
end
end

module SynchronousHttpRequest
Copy link
Contributor

@Samze Samze Nov 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question/non-blocking I see SynchronousHttpRequest is used https://github.com/cloudfoundry/cloud_controller_ng/blob/main/app/models/services/managed_service_instance.rb#L99 but is requester method called elsewhere? Quick grep would suggest maybe not.

rg requester
src/code.cloudfoundry.org/cc-uploader/handlers/upload_build_artifacts/upload_build_artifacts_test.go
131:		Context("when the requester (client) goes away", func() {

src/code.cloudfoundry.org/cc-uploader/handlers/upload_droplet/upload_droplet_test.go
178:		Context("when the requester (client) goes away", func() {

src/cloud_controller_ng/app/models/services/managed_service_instance.rb
98:    def requester

src/github.com/cloudfoundry/blobstore_url_signer/server/handler_test.go
49:			It("writes the signed URL back to requester", func() {
74:			It("writes the signed URL back to requester", func() {

Also, it seems like all of the lib/vcap/services/api stuff is from the very old v1 service broker contract, is that even used anywhere?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't think about it too hard at the time because with this PR I was trying to unblock the Ruby 3.3 upgrade, but I think you're right. I made another PR that removes the other old services code to keep the priority/purpose of this PR distinct: #4085

@sethboyles sethboyles merged commit 75ba025 into main Nov 11, 2024
8 checks passed
@sethboyles sethboyles deleted the remove-em-http-request branch November 11, 2024 23:35
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Nov 12, 2024
Changes in cloud_controller_ng:

- Remove unused gems
    PR: cloudfoundry/cloud_controller_ng#4073
    Author: Seth Boyles <[email protected]>

- Remove unused em-http-request
    PR: cloudfoundry/cloud_controller_ng#4069
    Author: Seth Boyles <[email protected]>
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