Lorawan-stack: دعم متغير مخصص في عناوين URL لبرنامج Webhook

بناءً على شكل الحقول ، أظن أنك كنت تخطط لتنفيذ ذلك باستخدام text/template . إذا كان الأمر كذلك بالفعل ، فالرجاء الانتباه إلى حقيقة أن تحليل القوالب وتنفيذها (خاصةً إذا لم يكن عنوان URL الأساسي فحسب ، بل المسار أيضًا) على كل ارتباط صاعد يمكن أن يكون مكثفًا للغاية للموارد.
تأكد أيضًا من استخدام بنية محدودة جدًا سيتم تنفيذ القالب بها. استخدم الأنواع الأولية فقط (لذا قم بتحويل جميع EUIs / DevAddr إلى سلاسل) ، وتجنب المؤشرات.

أعتقد أنه يمكننا تخزين القالب نفسه مؤقتًا باستخدام معرف Webhook + updated_at لتجنب إعادة تجميع القالب في كل مرة. كما أن الأمر كذلك بالفعل هو أن السلاسل فقط يجب أن يتم تمريرها إلى القالب نفسه (وأنا أفضل ألا نستخدم أي شيء آخر خارج المعرفات).

أعتقد خيارين.

• استخدم text/template ببنية بسيطة جدًا لحالة الاستخدام هذه. أي ، لا *ttnpb.ApplicationUp لكن WebhookTemplateFields بالأشياء التي ندعمها فقط
• استخدم استبدال السلسلة في الحقول المعروفة. بسرعة فائقة ، نفس النتيجة

إذا ذهبنا إلى الثانية ، فمن المحتمل ألا نستخدم بناء جملة نموذج Go على الإطلاق ، لأن ذلك من شأنه أن يربك الناس فقط (لماذا النقطة؟ هل هي مبطنة بالمسافات؟).

إذا أردنا الاحتفاظ بقواعد التحقق من الصحة كما هي ، فيمكننا التفكير في استخدام عناصر نائبة مثل $app_id . بهذه الطريقة لن تكون العناصر النائبة صالحة في المضيف (لأننا لا نريد ذلك) ، لكنها ستكون صالحة في المسار (أعتقد أن $ مسموح به في مسارات URL ، نحتاج إلى التحقق من ذلك ).

$ صالح كجزء من المضيف والمسار وفقًا لقسم RFC3986 2.2 . مع ما يقال ، لسنا مجبرين على تحليل الجزء المضيف من url.URL .

الصيغة القياسية (وفقًا لـ RFC 6570 ، انظر 1.2 و 2.2) لقوالب URI هي {[operator]variable-list} . نظرًا لأننا نعمل في منطقة ساخنة ، أعتقد أن هذا مبرر جيد للتخلي عن المشغلين (لأنهم سيتطلبون معالجة موسعة) ، والسماح بمتغير واحد فقط. هذا يسمح لنا باستخدام string.ReplaceAll بدون إعادة اختراع العجلة.

ماذا تقصد ب "السماح بمتغير واحد"؟

أعتقد أيضًا أن تجربة المستخدم من الأفضل عدم استخدام text/template ، بصرف النظر عن الأداء. أنا بخير مع أسماء المتغيرات بنمط PHP.

يسمح المعيار المذكور أعلاه بتعبيرات مثل {appEUI,devEUI} ، والتي لن تسمح لنا باستخدام استبدال سلسلة بسيطة. لاحظ أنني لم أمنع تعبيرات مثل {appEUI}/{devEUI} ، والتي تحتوي على متغير واحد فقط لكل لقطة ، وهي جيدة تمامًا.

ماذا عن هذا؟

هذه الحزمة هي حزمة داخلية ولا يمكن استيرادها خارج الحزمة الرئيسية ، ولكن هذا يجب أن يفعل ذلك.
تحرير: تعرض الحزمة الرئيسية الوظيفة أيضًا.