Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Update to new pseudo obfuscated chunks url #18

Merged
merged 3 commits into from
Aug 30, 2019
Merged

Update to new pseudo obfuscated chunks url #18

merged 3 commits into from
Aug 30, 2019

Conversation

jschwindt
Copy link
Contributor

@jschwindt jschwindt commented Jul 19, 2019

Desde hoy noté que RadioCut cambió el formato de la url de la API con la que se obtienen los chunks. Este PR soluciona eso. Lo probé sólo con python3, pero debería funcionar también con 2.

radiocut/__init__.py Outdated Show resolved Hide resolved
@luuuciano
Copy link

Desde hoy noté que RadioCut cambió el formato de la url de la API con la que se obtienen los chunks. Este PR soluciona eso. Lo probé sólo con python3, pero debería funcionar también con 2.

Puede ser que esto no ande bien en 2.7? hice los cambios y me sale esto:

$ radiocut https://radiocut.fm/audiocut/editorial-12-08-2019-macri-nunca-mas-pablo-ramos/
Traceback (most recent call last):
File "/usr/local/bin/radiocut", line 11, in
sys.exit(main())
File "/usr/local/lib/python2.7/dist-packages/radiocut/init.py", line 202, in main
audioclips = [get_audiocut(url, verbose, duration) for url in urls]
File "/usr/local/lib/python2.7/dist-packages/radiocut/init.py", line 67, in get_audiocut
chunks_url = get_chunks_url(base_url, station, start_folder)
File "/usr/local/lib/python2.7/dist-packages/radiocut/init.py", line 44, in get_chunks_url
code = base64.b64encode(bytes('win{}|{}dow'.format(station, start_folder), 'ascii'))
TypeError: str() takes at most 1 argument (2 given)

@jschwindt
Copy link
Contributor Author

@luuuciano lo acabo de probar con el fork que hice yo y funciona bien. Podés instalarlo de la siguiente forma desde mi repositorio:

pip install git+https://github.com/jschwindt/radiocut_downloader.git

@luuuciano
Copy link

pip install git+https://github.com/jschwindt/radiocut_downloader.git

Grande Juan! usé ese y sí, ya anda... no me avivé de copiar mis cambios a algún lado a ver en qué le había pifiado! jajaj...
Muchas gracias

Beers away!

@gnarvaja
Copy link

gnarvaja commented Aug 29, 2019 via email

@jschwindt
Copy link
Contributor Author

Y qué te parece que es? la concha de tu madre...

@mgaitan hizo este repositorio hace años y me puteás a mí por un PR? Capaz que si me pedís amablemente que no publique estas cosas lo borro y listo...

@mgaitan
Copy link
Owner

mgaitan commented Aug 29, 2019

posta @gnarvaja, esto nunca pretendió joder a radiocut. Entendemos que el nicho de usuarios es totalmente marginal y nunca osé armar clientes para windows, ni mucha doc justamente para mantenerlo asi, algo "para entendidos" que no afecte el negocio de ustedes.

Si ahora está afectando, decinos (o me tenes a un tuit de distancia) y dejamos de mantener esto y listo, pero no te engranés al pedo.

@gnarvaja
Copy link

gnarvaja commented Aug 30, 2019 via email

@jschwindt
Copy link
Contributor Author

Me sensibilizó el mensaje de alguien que basa su negocio en tomar "prestado" contenido producido por terceros (capaz sin permiso) y se ofende cuando a él le hacen lo mismo. Borro el repositorio de mi lado. La mejor de las suertes.

@mgaitan mgaitan merged commit 7822044 into mgaitan:master Aug 30, 2019
Repository owner locked as too heated and limited conversation to collaborators Aug 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants