From d44713b17990fb11230b3c5db81742aecb1f101e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20M=C3=BCller?= <webmaster@am-wd.de>
Date: Fri, 15 Nov 2024 17:16:37 +0100
Subject: [PATCH] Added missing priority on DNS record

---
 .gitlab-ci.yml                                  |  2 +-
 CHANGELOG.md                                    | 17 +++++++++++++++++
 .../Filters/ListDnsRecordsFilter.cs             |  4 ++--
 .../Cloudflare.Zones/Filters/ListZonesFilter.cs | 11 ++---------
 Extensions/Cloudflare.Zones/Models/DnsRecord.cs | 10 ++++++++++
 5 files changed, 32 insertions(+), 12 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 0778d47..d04e242 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -18,7 +18,7 @@ default-build:
     - lnx
     - 64bit
   rules:
-  - if: $CI_COMMIT_TAG == null
+    - if: $CI_COMMIT_TAG == null
   script:
     - dotnet build -c Debug --nologo
     - mkdir ./artifacts
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 1021240..7ef0eda 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1 +1,18 @@
 # Changelog
+
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [Unreleased]
+
+
+### Added
+
+- `Cloudflare` with basic functionality to communicate with Cloudflare's API
+- `Cloudflare.Zones` extending the core package with specific methods to manage Cloudflare's DNS zones
+
+
+
+[Unreleased]: https://github.com/AM-WD/cloudflare-api
diff --git a/Extensions/Cloudflare.Zones/Filters/ListDnsRecordsFilter.cs b/Extensions/Cloudflare.Zones/Filters/ListDnsRecordsFilter.cs
index 4276f09..d5f1eeb 100644
--- a/Extensions/Cloudflare.Zones/Filters/ListDnsRecordsFilter.cs
+++ b/Extensions/Cloudflare.Zones/Filters/ListDnsRecordsFilter.cs
@@ -148,13 +148,13 @@ namespace AMWD.Net.Api.Cloudflare.Zones
 		/// <summary>
 		/// Page number of paginated results.
 		/// </summary>
-		// >= 1
+		/// <value>1 &lt;= X</value>
 		public int? Page { get; set; }
 
 		/// <summary>
 		/// Number of DNS records per page.
 		/// </summary>
-		// >= 1 <= 5_000_000
+		/// <value>1 &lt;= X &lt;= 5,000,000</value>
 		public int? PerPage { get; set; }
 
 		/// <summary>
diff --git a/Extensions/Cloudflare.Zones/Filters/ListZonesFilter.cs b/Extensions/Cloudflare.Zones/Filters/ListZonesFilter.cs
index 1ae7329..acc50d1 100644
--- a/Extensions/Cloudflare.Zones/Filters/ListZonesFilter.cs
+++ b/Extensions/Cloudflare.Zones/Filters/ListZonesFilter.cs
@@ -11,7 +11,6 @@ namespace AMWD.Net.Api.Cloudflare.Zones
 		/// <summary>
 		/// An account ID.
 		/// </summary>
-		/// <value>account.id</value>
 		public string? AccountId { get; set; }
 
 		/// <summary>
@@ -32,19 +31,16 @@ namespace AMWD.Net.Api.Cloudflare.Zones
 		/// </remarks>
 		/// <example>Dev Account</example>
 		/// <example>contains:Test</example>
-		/// <value>account.name</value>
 		public string? AccountName { get; set; }
 
 		/// <summary>
 		/// Direction to order zones.
 		/// </summary>
-		/// <value>direction</value>
 		public SortDirection? Direction { get; set; }
 
 		/// <summary>
 		/// Whether to match all search requirements or at least one (any).
 		/// </summary>
-		/// <value>match</value>
 		public FilterMatchType? Match { get; set; }
 
 		/// <summary>
@@ -67,31 +63,28 @@ namespace AMWD.Net.Api.Cloudflare.Zones
 		/// <example>contains:.org</example>
 		/// <example>ends_with:arpa</example>
 		/// <example>starts_with:dev</example>
-		/// <value>name</value>
 		public string? Name { get; set; }
 
 		/// <summary>
 		/// Field to order zones by.
 		/// </summary>
-		/// <value>order</value>
 		public ZonesOrderBy? OrderBy { get; set; }
 
 		/// <summary>
 		/// Page number of paginated results.
 		/// </summary>
-		/// <value>page</value>
+		/// <value>1 &lt;= X</value>
 		public int? Page { get; set; }
 
 		/// <summary>
 		/// Number of zones per page.
 		/// </summary>
-		/// <value>per_page</value>
+		/// <value>5 &lt;= X &lt;= 50</value>
 		public int? PerPage { get; set; }
 
 		/// <summary>
 		/// A zone status.
 		/// </summary>
-		/// <value>status</value>
 		public ZoneStatus? Status { get; set; }
 
 		/// <inheritdoc />
diff --git a/Extensions/Cloudflare.Zones/Models/DnsRecord.cs b/Extensions/Cloudflare.Zones/Models/DnsRecord.cs
index d6cd97a..9c8f9bc 100644
--- a/Extensions/Cloudflare.Zones/Models/DnsRecord.cs
+++ b/Extensions/Cloudflare.Zones/Models/DnsRecord.cs
@@ -46,6 +46,16 @@ namespace AMWD.Net.Api.Cloudflare.Zones
 		[JsonProperty("content")]
 		public string? Content { get; set; }
 
+		/// <summary>
+		/// The priority.
+		/// </summary>
+		/// <remarks>
+		/// Required for <see cref="DnsRecordType.Mx"/>, <see cref="DnsRecordType.Srv"/> and <see cref="DnsRecordType.Uri"/> records; unused by other record types.
+		/// Records with lower priorities are preferred.
+		/// </remarks>
+		[JsonProperty("priority")]
+		public ushort? Priority { get; set; }
+
 		/// <summary>
 		/// Components associated with the record.
 		/// </summary>