Skip to content
This repository has been archived by the owner on Feb 10, 2022. It is now read-only.

Access Token en QueryString #102

Open
daniel-enviopack opened this issue Jan 12, 2021 · 21 comments
Open

Access Token en QueryString #102

daniel-enviopack opened this issue Jan 12, 2021 · 21 comments

Comments

@daniel-enviopack
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

me sumo, si alguien tiene modificaciones son agradecidas :s

@francosabolo
Copy link

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
Copy link

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

@katrazka
Copy link

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
Copy link

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

@ingfdoaguirre
Copy link

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
Copy link

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
Copy link

@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
Copy link

katrazka commented Mar 24, 2021

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
Copy link

@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
Copy link

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
Copy link

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
Copy link

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
Copy link

katrazka commented Mar 24, 2021 via email

@andresmaiden
Copy link

andresmaiden commented Mar 24, 2021 via email

@katrazka
Copy link

katrazka commented Mar 24, 2021 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants