Implementando el registro

Clase 11 de 26 • Curso de Go Avanzado: REST y WebSockets

Eddy Paz

student•

Considerando que es un curso avanzado de go, veo bastante bien que haya incluido docker para la creacion de la bbdd y asi no cargar el curso con un video de instalacion de bbdd o de introduccion a docker, ya que al ser golang no deberia de mesclarse, queda como tarea para los estudiantes aprender a instalar docker, lo cual es sumamente sencillo y luego solo correr el dockerfile y no se tendria ningun problema, utilizar docker para el levantado en base de datos me parece muy novedoso y util para este tipo de cursos Excelente!

Jose Leon Faneite

student•

package handlers

import (
	"encoding/json"
	"net/http"

	"github.com/segmentio/ksuid"
	"platzi.com/models"
	"platzi.com/repository"
	"platzi.com/server"
)

type SignUpRequest struct {
	Email    string `json:"email"`
	Password string `json:"password"`
}

type SignUpResponse struct {
	Id    string `json:"id"`
	Email string `json:"email"`
}

func SignUpHandler(s server.Server) http.HandlerFunc {
	return func(w http.ResponseWriter, r *http.Request) {
		var request = SignUpRequest{}
		err := json.NewDecoder(r.Body).Decode(&request)
		if err != nil {
			http.Error(w, err.Error(), http.StatusBadRequest)
			return
		}

		id, err := ksuid.NewRandom()
		if err != nil {
			http.Error(w, err.Error(), http.StatusInternalServerError)
			return
		}

		var user = models.User{
			Email: request.Email,
			Pass:  request.Password,
			ID:    id.String(),
		}

		err = repository.InsertUser(r.Context(), &user) // se inyecta la implementacion de la interfaz en tiempo de ejecucion
		if err != nil {
			http.Error(w, err.Error(), http.StatusInternalServerError)
			return
		}

		w.Header().Set("Content-Type", "application/json")
		json.NewEncoder(w).Encode(SignUpResponse{
			Id:    user.ID,
			Email: user.Email,
		})
	}
}

<code> ```

Facundo Yuffrida

student•

No me parece que sea correcto meter aquí Docker sin ninguna clase de explicación. Esto se suele hacer mucho en la enseñanza de tecnologías de creación de software y hay que ser consciente para poder detenerlo. Lo que se explica en un curso debe estar previamente introducido, ya sea en el propio curso, en un curso anterior requerido para entender este o ser un conocimiento requerido al iniciar el curso. Docker acá no cumple con ninguno de estos items. Quien no lo entiende, tal vez piensa que tiene que hacerse un curso de Docker previamente, y la realidad es que no tiene nada que ver con este curso, no debería tener que aprenderlo. Punto negativo para el curso.

Nicolás Sañudo

student•

Hola Facundo. Estaba revisando lo que dices, y en la página principal del curso tiene un listado de requisitos previos donde figura Docker.

No puedo asegurar si lo implementaron antes o después de tu comentario. Si noto que el curso forma parte de la ruta Desarrollo Backend con Go, y antés de este curso figura el curso de Docker. Supongo que el curso está pensado para quienes sigan esa ruta.

Felipe Hernández

student•

Está listado en los requisitos del curso. Y de todas formas no está de más aprenderlo, ya que Docker llegó para quedarse en casi todos los ambientes de producción con servicios REST, y tarde o temprano vas a tener que aprenderlo.

José Alberto Mata Mena

student•

Creo que hay varías formas de resolver el problema y ya dependerá de nuestro equipo o como le qusieramos hacer. Personalmente lo haría verificando a nivel de codigo si existe un usuario con ese correo, si existe que retorne error, sino que proceda a hacer el insert.

Otra forma podría ser creando un Trigger a nivel de DB y verificar en la propia base de datos si existe un registro con el correo, y si es así que haga un rollback de la transacción.

Diego Posada

student•

Hola alguien sabe depronto cual es la ventaja de utilizar la dependencia que indica el docente https://github.com/segmentio/ksuid vs tener el campo autoincrementable definido en la base de datos?

Jonatan Camilo Bonilla Malaver

student•

Hola, no es que sea una ventaja o desventaja; lo que pasa es que el profesor en vez de usar un campo entero auto incremental, lo que hizo fue definirlo como un string y la dependencia lo que te ayuda es a generar un UUID o Universal Unique ID que es un string similar al RFC 4122 UUID, lo cual puedes buscar mas información al respecto. El que haya elegido usar un string vs un entero depende mucho de lo que necesitas en tu modelo de datos, por lo que para este ejemplo el profesor decidió eso, así que no te preocupes. Depende de lo que estés haciendo, de tu aplicación y los requerimientos que esta tenga.

Espero haberte ayudado, un saludo.

Jorge Vidoni

student•

@DiegoAll la utilización de la librería es solo para obtener la utilizad de generar códigos aleatorios con baja posibilidad de colisiones.

Lo que tenes que analizar es la ventajas/desventajes de utilizar id aleatorios vs secuenciales.

La utilización de id secuenciales tiene la ventaja que es muy simple de implementar y te evita lidear con el problema de las colisiones, pero tiene como gran desventaja que es más inseguro. En especial en APIs que se exponen al mundo.

Para ejemplificar esto imagina que desarrollas un API que le permita a tus clientes ver el detalle de los productos que vendes. Pero a vos solo te interesa que solo puedan acceder a determinados productos que le vas a compartir.

La URL de tu API es la siguiente .../products/{product-id}, y tiene una implementación de id secuencial, con esta implementación tus clientes pueden empezar a recorrer toda tu base de datos con un pequeño script que empiece a incrementar en uno el id del producto.

Esto ultimo se hace mucho más complicado cuando los id se generan de forma aleatoria. Aunque si tu api no va a estar expuesta al publico posiblemente no te convenga lidiar con los problemas de utilizar un id aleatorio

brandon james grimaldo moscote

student•

go get github.com/lib/pq

brandon james grimaldo moscote

student•

go get github.com/segmentio/ksuid

Diego Posada

student•

Alguien sabe por que en el ejercicio de todo el curso el docente no utiliza un controller?

¿Tiene que ver con el patrón repositorio?

Jonatan Camilo Bonilla Malaver

student•

Hola Diego, no tiene nada que ver con el patrón repositorio, de hecho el profesor lo ha estado usando y es que en go la convención o el equivalente a controllers son los handlers.

Julio Cano López

student•

En algún momento escuché hablar de esto, lo comparto por acá para que tengan en cuenta al momento de hacer sus diseños de APIS, es muy funcional saberlo para definir de forma correcta responsabilidades de métodos. Modelo de madurez de richardson [Enlace](https://martinfowler.com/articles/richardsonMaturityModel.html)

Kevin Bueno

student•

Pienso que al SignUpHandler debemos agregarle un código http 201 CREATED

w.WriteHeader(http.StatusCreated)

Fernando Cordero

student•

eso es correcto amigo.