-
Notifications
You must be signed in to change notification settings - Fork 265
Access Token en QueryString #102
Comments
Hola. Esta bastante facil, pero en la misma clase dice que fue autogenerada y que no debe modificarse. Si podemos hacer alguna modificacion, con gusto la hago, pero ya no se si esta permitido. |
Ponga en código e hace un Pull Request |
Hola, me sumo a este tema, saben si se corrigió esto en la sdk? estamos a 8 días de que no se admitan mas los request sin el token en la cabecera. |
Hola a como se ve, el SDK ya no lo van a mantener, asi que ya queda en nosotros hacer la modificacion. Estoy por hacerlo yo, hice un fork en mi profile de github, ahi subire el sdk modificado, espero manana ya tenerlo listo. Saludos. |
Buenas, recién me había dado cuenta de lo de la SDK. Ya me estoy poniendo en camino a hacer modificaciones aunque me parece que aca nos vamos a ir directamente por quitar la SDK lo mas posible y hacer cURLs para los get, post y put hacia ML. gracias por contestar, Saludos. |
me sumo, si alguien tiene modificaciones son agradecidas :s |
Buenas, no llegue a probarlo aún pero creo que los cambios serían los que dejo en esta PR https://github.com/mercadolibre/php-sdk/pull/106/files |
Se ve bien el codigo en ese PR, voy a probarlo a ver que tal |
Hice este cambio y funciono, si creen que esta mal o falta algo avisen, y si hay un foro donde podamos hacernos consultas entre nosotros avisen tambien porque nos dejaron en banda desde que cerraron el foro "oficial" lib>Api>RestClientApi.php linea 640 (lo comente) Linea 702 - Agregue antes de $headers = array_merge( //Agregado para 1/4/2021 $defaultHeaders['Authorization'] = "Bearer $access_token"; |
@ingfdoaguirre @katrazka Agregué un cambio en el PR y está funcionando también. |
Gracias por la ayuda amigos. Pues que se haga lo del foro, podemos crear en algun sitio gratuito de foros, para ayudarnos entre nosotros y no estar solos. |
si conocen alguno que ofresca algo simple procedan y me sumo |
En la linea que esta esto: yo lo comente porque es la forma actual de pasar el token, por lo que estarias pasandolo 2 veces y hay que ver cual toma, por eso yo lo comente para verificar que el agregado funcione. Fijate si sacandolo funciona. |
Aclaro: Soy "programador" autodidacta con poca experiencia en php basica, no tengo la costumbre de publicar cosas en ingles, y usar las palabras que se suelen usar en el medio, lo mio es basico pero me funciona |
Si, creo que estabas viendo una versión anterior ya quite esa parte :) gracias! |
En otros proyectos de repos tenemos grupos de Google Groups, es muy cómodo y además posiciona muy bien en Google (obvio) para los que buscan temas relacionados, por lo que la comunidad va creciendo bien. Si les parece lo creo y lo comparto acá para que podamos trabajar mejor sobre el plugin. |
Buenos días a todos. |
Me parece excelente, yo te doy mi apoyo. Algo como "Comunidad Developers ML" para que no nos lo den de baja por incluir la palabra Mercadolibre o algo asi |
dale
Juanma!
El mié, 24 mar 2021 a las 9:56, Andres Misiak ***@***.***>)
escribió:
… Gracias por la ayuda amigos. Pues que se haga lo del foro, podemos crear
en algun sitio gratuito de foros, para ayudarnos entre nosotros y no estar
solos.
si conocen alguno que ofresca algo simple procedan y me sumo
En otros proyectos de repos tenemos grupos de Google Groups, es muy cómodo
y además posiciona muy bien en Google (obvio) para los que buscan temas
relacionados, por lo que la comunidad va creciendo bien. Si les parece lo
creo y lo comparto acá para que podamos trabajar mejor sobre el plugin.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#102 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADRGN2DBYSABUTU2BCGFYPTTFHOPPANCNFSM4V7K4YPA>
.
|
Para ya cerrar el issue e ir a un foro, les paso el que acabo de crear,
ojalá funcione:
https://groups.google.com/g/developers-php-mercadolibre
Había otro en el que estuve hace unos años, pero está muerto, hace 6 años
que nadie escribe (https://groups.google.com/g/es-meli-developers), el tema
es que me pareció importante enfocar en PHP que es el lenguaje en el que
usamos este SDK.
Avisen si alguien más quiere ser administrador y lo pongo, así no estoy
solo.
Saludos.
*Andres Misiak*
El mié, 24 de mar. de 2021 a la(s) 12:53, katrazka ***@***.***)
escribió:
… dale
Juanma!
El mié, 24 mar 2021 a las 9:56, Andres Misiak ***@***.***>)
escribió:
> Gracias por la ayuda amigos. Pues que se haga lo del foro, podemos crear
> en algun sitio gratuito de foros, para ayudarnos entre nosotros y no
estar
> solos.
>
> si conocen alguno que ofresca algo simple procedan y me sumo
>
> En otros proyectos de repos tenemos grupos de Google Groups, es muy
cómodo
> y además posiciona muy bien en Google (obvio) para los que buscan temas
> relacionados, por lo que la comunidad va creciendo bien. Si les parece lo
> creo y lo comparto acá para que podamos trabajar mejor sobre el plugin.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <
#102 (comment)
>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/ADRGN2DBYSABUTU2BCGFYPTTFHOPPANCNFSM4V7K4YPA
>
> .
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#102 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAYBM53XA2L2ETPAGSVLSXLTFIDF7ANCNFSM4V7K4YPA>
.
|
ahi me sume, no te digo de ser admin porque soy bastante queso jaja
Juanma!
El mié, 24 mar 2021 a las 18:19, Andres Misiak ***@***.***>)
escribió:
… Para ya cerrar el issue e ir a un foro, les paso el que acabo de crear,
ojalá funcione:
https://groups.google.com/g/developers-php-mercadolibre
Había otro en el que estuve hace unos años, pero está muerto, hace 6 años
que nadie escribe (https://groups.google.com/g/es-meli-developers), el
tema
es que me pareció importante enfocar en PHP que es el lenguaje en el que
usamos este SDK.
Avisen si alguien más quiere ser administrador y lo pongo, así no estoy
solo.
Saludos.
*Andres Misiak*
El mié, 24 de mar. de 2021 a la(s) 12:53, katrazka ***@***.***)
escribió:
> dale
>
>
> Juanma!
>
>
> El mié, 24 mar 2021 a las 9:56, Andres Misiak ***@***.***>)
> escribió:
>
> > Gracias por la ayuda amigos. Pues que se haga lo del foro, podemos
crear
> > en algun sitio gratuito de foros, para ayudarnos entre nosotros y no
> estar
> > solos.
> >
> > si conocen alguno que ofresca algo simple procedan y me sumo
> >
> > En otros proyectos de repos tenemos grupos de Google Groups, es muy
> cómodo
> > y además posiciona muy bien en Google (obvio) para los que buscan temas
> > relacionados, por lo que la comunidad va creciendo bien. Si les parece
lo
> > creo y lo comparto acá para que podamos trabajar mejor sobre el plugin.
> >
> > —
> > You are receiving this because you were mentioned.
> > Reply to this email directly, view it on GitHub
> > <
>
#102 (comment)
> >,
> > or unsubscribe
> > <
>
https://github.com/notifications/unsubscribe-auth/ADRGN2DBYSABUTU2BCGFYPTTFHOPPANCNFSM4V7K4YPA
> >
> > .
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <
#102 (comment)
>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/AAYBM53XA2L2ETPAGSVLSXLTFIDF7ANCNFSM4V7K4YPA
>
> .
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#102 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADRGN2AR2PDUK6LLDX3WAP3TFJJM5ANCNFSM4V7K4YPA>
.
|
Llego una alerta propia de Mercadolibre indicando:
A partir del 1 de abril será obligatorio enviar el access token en el header de las llamadas a nuestra API
Revisando la SDK, la propia SDK de MELI no cumple con este requisito.
Necesitamos que se actualice la misma dado que estamos intengrados via SDK asimismo por requisito propio de MELI.
Saludos.
The text was updated successfully, but these errors were encountered: