search

mercadolibre / php-sdk

MercadoLibre's PHP SDK
185 stars 266 forks source link

Access Token en QueryString #102

Open daniel-enviopack opened 3 years ago

daniel-enviopack commented 3 years ago

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.

ingfdoaguirre commented 3 years ago

Hola. Tenemos hasta Abril para adaptarnos. Estaba viendo que la funcion que hace el request es resourceGetRequest en la clase RestApiClient donde en lugar de armar el query param, es pasar el token al momento de armar el header.

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.

rodrigojob commented 3 years ago

Necesitamos que se actualice la misma dado que estamos intengrados via SDK asimismo por requisito propio de MELI.

Ponga en código e hace un Pull Request

arielrando commented 3 years ago

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.

ingfdoaguirre commented 3 years ago

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.

arielrando commented 3 years ago

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.

katrazka commented 3 years ago

me sumo, si alguien tiene modificaciones son agradecidas :s

francosabolo commented 3 years ago

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

ingfdoaguirre commented 3 years ago

Se ve bien el codigo en ese PR, voy a probarlo a ver que tal

katrazka commented 3 years ago

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) // $queryParams['access_token'] = $access_token;

Linea 702 - Agregue antes de $headers = array_merge(

//Agregado para 1/4/2021

$defaultHeaders['Authorization'] = "Bearer $access_token"; //Agregado para 1/4/2021

francosabolo commented 3 years ago

@ingfdoaguirre @katrazka Agregué un cambio en el PR y está funcionando también. https://github.com/mercadolibre/php-sdk/pull/106/files

ingfdoaguirre commented 3 years ago

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.

katrazka commented 3 years ago

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

katrazka commented 3 years ago

@ingfdoaguirre @katrazka Agregué un cambio en el PR y está funcionando también. https://github.com/mercadolibre/php-sdk/pull/106/files

En la linea que esta esto: $queryParams['access_token'] = $access_token;

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.

katrazka commented 3 years ago

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

francosabolo commented 3 years ago

@ingfdoaguirre @katrazka Agregué un cambio en el PR y está funcionando también. https://github.com/mercadolibre/php-sdk/pull/106/files

En la linea que esta esto: $queryParams['access_token'] = $access_token;

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.

Si, creo que estabas viendo una versión anterior ya quite esa parte :) gracias!

andresmaiden commented 3 years ago

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.

gioramies commented 3 years ago

Buenos días a todos. Recién voy a consumir el API de Mercado Libre en una aplicación con PHP principalmente para Inmuebles. ¿Qué me recomendarían dado que ya esta librería está Deprecated? Muchas gracias por sus recomendaciones.

ingfdoaguirre commented 3 years ago

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.

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

katrazka commented 3 years ago

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 https://github.com/mercadolibre/php-sdk/issues/102#issuecomment-805797134, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADRGN2DBYSABUTU2BCGFYPTTFHOPPANCNFSM4V7K4YPA .

andresmaiden commented 3 years ago

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 < https://github.com/mercadolibre/php-sdk/issues/102#issuecomment-805797134 , 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 https://github.com/mercadolibre/php-sdk/issues/102#issuecomment-805942931, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYBM53XA2L2ETPAGSVLSXLTFIDF7ANCNFSM4V7K4YPA .

katrazka commented 3 years ago

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 <

https://github.com/mercadolibre/php-sdk/issues/102#issuecomment-805797134

, 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 < https://github.com/mercadolibre/php-sdk/issues/102#issuecomment-805942931 , 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 https://github.com/mercadolibre/php-sdk/issues/102#issuecomment-806192931, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADRGN2AR2PDUK6LLDX3WAP3TFJJM5ANCNFSM4V7K4YPA .