Los nombres de las variables son intelegibles
Closed this issue · 9 comments
Estoy intentando crear una versión en Python de la biblioteca pero los nombres de las variables hacen mucho más dificil entender que está haciendo el código, por ejemplo:
function crearDatospago($sp821fb9, $sp37cd46=null)¿Qué son los parámetros? Es un misterio. ¿Están ofuscados a propósito? Deberían cambiarlos a algo como esto:
function crearDatospago($datos_venta, $extra=null)(y estoy adivinando que esos son los nombres correctos)
Hola JP,
la libreria de PHP esta ofuscada.
Escribeme a mi email carlos.morales@culqi.com, para tratar el tema de la libreria que quieres desarrollar y poder guiarte, un abrazo.
Gracias por la oferta, Carlos.
Sin embargo no tiene ningún sentido ofuscar su código en este caso:
- Es una biblioteca para conectarse a su servicio, el que me imagino quieren que la gente use. ¿Acaso pretenden tener que escribir ustedes mismos adaptadores para cada lenguaje actual y futuro?
- Es más dificil decifrarlo pero no es, ni de lejos, imposible. Cualquiera con el tiempo disponible podría hacerlo. Entonces no protege realmente nada. Si alguien busca un bug para explotarlo, lo decifrará de todos modos.
- Mientras más dificil sea de interpretar, más improbable es que alguien bienintencionado encuentre un bug y los proteja de (2)
- Si llego a escribir una versión en Python, las variables tendrán nombres amigables y estará documentado de la forma más clara posible. Entonces los demás podrían usarla como referencia.
👍 a información no ofuscada :)
Hola Juan Pablo,
las librerías que hemos puesto a disposición (PHP y Java) se conectan a una
API que aún no esta documentada al 100%.
Esta API servirá para que los desarrolladores puedan hacer su propia
librería en el lenguaje que dominen.
La estrategia actual, es que si vas a desarrollar en un lenguaje diferente
a las 2 librerías que hemos publicado nos lo comuniques para darte el Draft
del API.
El dic. 30, 2015 13:40, "Alvaro Pereyra Rabanal" notifications@github.com
escribió:
[image: 👍] a información no ofuscada :)
—
Reply to this email directly or view it on GitHub
#3 (comment).
Y si están en eso, sería muy genial que se refieran a su biblioteca correctamente como biblioteca y no "librería".
Yo entiendo que al trabajar con tarjetas y pagos se necesite ser bastante sensible con la información que hacen pública, pero Stripe también lo hace y no se dedica a ofuscar el código. De hecho, tienen 133 pull requests y no necesitan enviar correos para aceptar contribuciones.
Culqi está genial, pero creo que deberían relacionarse mejor con los devs (así como lo hace Conekta en México), porque al final eso nos beneficiará a todos.
Totalmente de acuerdo Gustavo.
Pero comprendenos que en estos momentos estamos trabajando en la
publicación del API.
El dic. 30, 2015 13:50, "Gustavo Leon" notifications@github.com escribió:
Yo entiendo que al trabajar con tarjetas y pagos se necesite ser bastante
sensible con la información que hacen pública, pero Stripe también lo hace
y no se dedica a ofuscar el código https://github.com/stripe/stripe-php.
De hecho, tienen 133 pull requests y no necesitan enviar correos para
aceptar contribuciones.Culqi está genial, pero creo que deberían relacionarse mejor con los devs
(así como lo hace Conekta https://github.com/conekta en México), porque
al final eso nos beneficiará a todos.—
Reply to this email directly or view it on GitHub
#3 (comment).
Entiendo eso, y de nuevo, me parece genial lo que hacen (deben ser los primeros en Perú y eso es admirable), pero: 1) las APIs no están escritas en piedra, y 2) una biblioteca con comentarios y phpDocumentor sería suficiente para muchos.
Éxitos :)
Gracias Gustavo por tu aporte y punto de vista, es valioso para nosotros.
El dic. 30, 2015 14:05, "Gustavo Leon" notifications@github.com escribió:
Entiendo eso, y de nuevo, me parece genial lo que hacen (deben ser los
primeros en Perú y eso es admirable), pero: 1) las APIs no están escritas
en piedra, y 2) una biblioteca con comentarios y phpDocumentor sería
suficiente para muchos.Éxitos :)
—
Reply to this email directly or view it on GitHub
#3 (comment).