Link, ClickAction Https Validation, before request.

Đang mở
  • Có gì không ổn?
#346 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
35/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
csharp
Lĩnh vực
api, backend

Hướng nghiên cứu

Bắt đầu tại WebpushFcmOptions.CopyAndValidate, điểm vào của việc xác thực được nêu trong issue, và so sánh hành vi của nó với AndroidNotification.ClickAction. Xác nhận với các maintainer xem có nên chấp nhận các liên kết HTTP hay việc xác thực nên có thể cấu hình hay không; được coi là hoàn tất khi hành vi đã thống nhất được triển khai và được bao phủ bởi các bài kiểm thử messaging liên quan.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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}");
Ngôn ngữ chính
C#
Star
407
Fork
147
Merge trung bình
2 ngày 10 giờ
Pull request đã merge (30 ngày)
12

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của firebase/firebase-admin-dotnet

Tất cả issue của firebase/firebase-admin-dotnet

Issue tương tự

Thêm issue về C#

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.