Link, ClickAction Https Validation, before request.

Abierto
  • ¿Algo no está bien?
#346 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
csharp
Área
api, backend

Línea de trabajo

Comienza en WebpushFcmOptions.CopyAndValidate, el punto de entrada de validación mostrado en el issue, y compara su comportamiento con AndroidNotification.ClickAction. Confirma con los mantenedores si deben aceptarse enlaces HTTP o si la validación debe ser configurable; se considera terminado cuando el comportamiento acordado esté implementado y cubierto por las pruebas de mensajería pertinentes.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

needs-triage
[REQUIRED] Step 2: Describe your environment
  • Operating System version: x64
  • Firebase SDK version: 2.3.0
  • Firebase Product: Cloud Messaging
  • .NET version: .NET 6.0
  • OS: Win10
[REQUIRED] Step 3: Describe the problem
Steps to reproduce:

I trying to send a push messaage with:
-Webpush data (FcmOptions.Link)
-Android data (Notification.ClickAction)

Link and ClickAction same, this is http links, not https, because this is my local development enviroment.
When i developing i want to send messages with http links that pointing on my local circuit wtihout https.

Before message sends, it's validating, and validation checks link only in webpush, Android click action does not checks, and if link not https throws exception with mesage:

System.ArgumentException: 'The link options should be a valid https url.'

image

I want to have a opportunity to disable this validation, but in best case scenario remove this validation from code, beacause i dowloaded source code, and used it instead of package, after removing https checks, and FCM succeffully processed this message, when i got it and clicked on message, my browser successfully opened on http - link.

this place in source code:

using System;
using Newtonsoft.Json;

namespace FirebaseAdmin.Messaging
{
    public sealed class WebpushFcmOptions
    {
        [JsonProperty("link")]
        public string Link { get; set; }

        internal WebpushFcmOptions CopyAndValidate()
        {
            var copy = new WebpushFcmOptions()
            {
                Link = this.Link,
            };

            if (copy.Link != null)
            {
	        // this https validation, that i want to remove
                if (!Uri.IsWellFormedUriString(copy.Link, UriKind.Absolute) || !copy.Link.StartsWith("https"))
                {
                    throw new ArgumentException("The link options should be a valid https url.");
                }
            }

            return copy;
        }
    }
}
Relevant Code:
using FirebaseAdmin;
using FirebaseAdmin.Messaging;
using Google.Apis.Auth.OAuth2;

var credential = GoogleCredential.FromFile("private-key.json");

FirebaseApp.Create(new AppOptions()
{
    Credential = credential
});

var registrationToken = "CLIENT_TOKEN_HERE";

var message = new Message()
{
    Token = registrationToken,
    Notification = new Notification()
    {
        Title = "Test title.",
        Body = "Test body.",
        
    },
    Webpush = new WebpushConfig()
    {
        FcmOptions = new WebpushFcmOptions()
        {
	    // http link that will be cheked
            Link = "http://127.0.0.1:5173"
        }
    },
    Android = new AndroidConfig()
    {
        Notification = new AndroidNotification()
        {
	    // this will not checked
            ClickAction = "http://127.0.0.1:5173"
        }
    }
};

var response = await FirebaseMessaging.DefaultInstance.SendAsync(message);

Console.WriteLine($"Successfully send message to {response}");
Lenguaje dominante
C#
Estrellas
408
Forks
147
Merge medio
1 d 3 h
PR fusionados (30 d)
10

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de firebase/firebase-admin-dotnet

Todos los issues de firebase/firebase-admin-dotnet

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.