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.

Sign up for GitHub

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

Jump to bottom

Runtime error in OTP 24 #140

Open
tobstarr opened this issue Jun 17, 2021 · 2 comments
Open

Runtime error in OTP 24 #140

tobstarr opened this issue Jun 17, 2021 · 2 comments

Comments

@tobstarr
Copy link

tobstarr commented Jun 17, 2021 •
edited
Loading

The library no longer works in OTP 24 as :crypto.hmac/3 was removed in favour over crypto:mac/4.
@tobstarr tobstarr changed the title Compile error in OTP 24 Runtime error in OTP 24 Jun 17, 2021
@tobstarr
Copy link
Author

Diving a little bit deeper, the issue seems to be caused by https://github.com/lexmag/oauther. Will try to file an issue there as well.

@tobstarr tobstarr mentioned this issue Jun 17, 2021
Closed
hashrocketeer added a commit to hashrocket/tilex that referenced this issue Aug 6, 2021
@hashrocketeer
Forces oauther library version that deals with OTP 24
3742764 
The newest versions of OTP do not have `:crypto.hmac/3` anymore and this
has broken TIL integration with Twitter since the last big app bump to
use OTP 24. `extwitter` library uses `oauther` which does not have
changed the `:crypto.mac/4` yet, so we're going to use this hot fix
branch for now.

For reference:
lexmag/oauther#22
parroty/extwitter#140
@hashrocketeer hashrocketeer mentioned this issue Aug 6, 2021
Merged
1 task
hashrocketeer added a commit to hashrocket/tilex that referenced this issue Aug 6, 2021
@hashrocketeer
Forces oauther library version that deals with OTP 24
58fd306 
The newest versions of OTP do not have `:crypto.hmac/3` anymore and this
has broken TIL integration with Twitter since the last big app bump to
use OTP 24. `extwitter` library uses `oauther` which does not have
changed the `:crypto.mac/4` yet, so we're going to use this hot fix
branch for now.

For reference:
lexmag/oauther#22
parroty/extwitter#140
hashrocketeer added a commit to hashrocket/tilex that referenced this issue Aug 6, 2021
@hashrocketeer
Forces oauther library version that deals with OTP 24
5f1da87 
The newest versions of OTP do not have `:crypto.hmac/3` anymore and this
has broken TIL integration with Twitter since the last big app bump to
use OTP 24. `extwitter` library uses `oauther` which does not have
changed the `:crypto.mac/4` yet, so we're going to use this hot fix
branch for now.

For reference:
lexmag/oauther#22
parroty/extwitter#140
@parroty
Copy link
Owner

parroty commented Aug 11, 2021

Thank you for the report and fixes on the upstreams. I'll be thinking to wait for the upstream to be merged at the moment (then update dependency), but if there's anything needs to be done, please let me know (if it takes longer, possibly refer to your repository, etc.).

hashrocketeer added a commit to hashrocket/tilex that referenced this issue Aug 24, 2021
@hashrocketeer
Forces oauther library version that deals with OTP 24
100fe91 
The newest versions of OTP do not have `:crypto.hmac/3` anymore and this
has broken TIL integration with Twitter since the last big app bump to
use OTP 24. `extwitter` library uses `oauther` which does not have
changed the `:crypto.mac/4` yet, so we're going to use this hot fix
branch for now.

For reference:
lexmag/oauther#22
parroty/extwitter#140
@parroty parroty mentioned this issue Dec 28, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tobstarr @parroty