Access Token en QueryString

[daniel-enviopack]

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]

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]

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]

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]

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]

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]

me sumo, si alguien tiene modificaciones son agradecidas :s

[francosabolo]

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]

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

[katrazka]

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]

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

[ingfdoaguirre]

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]

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]

@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]

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]

@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!