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

Local Jump Error Listing Recordings #663

Open
JYorston opened this issue Jun 16, 2023 · 3 comments
Open

Local Jump Error Listing Recordings #663

JYorston opened this issue Jun 16, 2023 · 3 comments
Labels
status: work in progress Twilio or the community is in the process of implementing type: bug bug in the library

Comments

@JYorston
Copy link
Contributor

JYorston commented Jun 16, 2023

Issue Summary

Upgrading from 5.56.0 to 6.1.0 has introduced a bug for us in the following code.

Downgrading back to 5.56.0 resolves the issue.

Steps to Reproduce

  1. Run the below code snippet on 6.1.0, local jump error exists.
  2. Rollback to 5.56.0, error does not occur.

Code Snippet

        client = Twilio::REST::Client.new(api_key_sid, api_key_secret, account_sid)
        recordings = client.video.v1.recordings.list(grouping_sid: ['SOME_ROOM_ID'],
                                                    limit: 20)

Exception/Log

LocalJumpError: no block given (yield)
from /Users/jamesyorston/.rbenv/versions/2.7.6/lib/ruby/gems/2.7.0/gems/twilio-ruby-6.0.1/lib/twilio-ruby/framework/serialize.rb:77:in `block in serialize_list'

Technical details:

  • twilio-ruby version: 6.1.0
  • ruby version: 2.7.6
@JYorston JYorston changed the title Logical Jump Error Local Jump Error Listing Recordings Jun 16, 2023
@JYorston
Copy link
Contributor Author

JYorston commented Jun 16, 2023

#665

A small change, it appears this block used to be passed in previous versions and was removed. I have added it back in.

This fixes the issue.

@AsabuHere
Copy link
Contributor

AsabuHere commented Jun 20, 2023

Hi @JYorston, Our team has reviewed and merged this PR. Please note that these are auto-generated files via our internal oai-generator, which would need modifications to incorporate this change in next generations. Tracked via DII-1038.

To unblock you in development usage, these changes are approved for now. Please wait till the next release for the fix to be available in sdk.

@AsabuHere AsabuHere added type: bug bug in the library status: ready for deploy code ready to be released in next deploy status: work in progress Twilio or the community is in the process of implementing and removed status: ready for deploy code ready to be released in next deploy labels Jun 20, 2023
@JYorston
Copy link
Contributor Author

@AsabuHere Great, thanks for the quick turn around!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: work in progress Twilio or the community is in the process of implementing type: bug bug in the library
Projects
None yet
Development

No branches or pull requests

2 participants