Put urls for service calls to Mail Service into a config file
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza en packagecontroller.cs y compara la configuración de URL existente de packageservice con la Mail Service URL codificada directamente que se muestra en el issue. Sigue cómo el valor configurado llega a las llamadas al servicio y, después, verifica que producción pueda proporcionar la URL mediante GitHub secrets y que no quede ninguna Mail Service URL codificada directamente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
issue in packagecontroller.cs
...
PackageServiceModel pkgService = await flurlPackageResponse.GetJsonAsync<PackageServiceModel>();
PackageResponseModel pkg = ServiceToResponseModelConverter.ConvertPackage(pkgService, await GetAllPersons(), await GetAllRooms());
Console.WriteLine("package route finished is " + pkg.RouteFinished);
if (pkg.RouteFinished)
{
//send email
await "https://mailservice20210603092014.azurewebsites.net/api/ArrivalMail?code=gYOUs9FO7WwwNXz2eSGtZM0AFxQl/RQvOJ4RF0uotwYLe7l/AIGGKg==".PostJsonAsync(pkg);
}
...
currently the urls for the calls to the serverless function mailservice is hardcoded
good practice would be to move the urls into a config file and take the value from the config file. Figure out a way to take the url from the config file. Note: in production, the url should be taken from GitHub secrets!
tip:
look at how the packageservice url has been set up as an example of how it's done
- Lenguaje dominante
- C#
- Estrellas
- 1
- Forks
- 1
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de FIPost/api-gateway
-
externe servicesAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
FIPost/api-gateway#41 ·
-
Investigation
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
FIPost/api-gateway#39 ·
-
Investigation
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
FIPost/api-gateway#38 ·
-
Investigation
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
FIPost/api-gateway#37 ·
-
bug good first issue
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
FIPost/api-gateway#35 ·
Todos los issues de FIPost/api-gateway
Issues similares
-
copilot documentation
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 2 días
-
[Rust][Flaky Test] multiple_deadlines_fire_in_order asserts a wall-clock gap instead of firing orderAbiertoCI/CD ⚒️ Flaky-tests 🐦
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
valkey-io/valkey-glide#7255 ·
Los mantenedores suelen responder en 3 días
-
bug good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
unoplatform/Uno.Core#99 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
aws/aws-dotnet-ai#75 ·