control description length <= 280

smart-contracts/assembly/contracts/dns/__tests__/dns.spec.ts

@@ -1,6 +1,7 @@
 import { contractOwnerKey, blackListKey } from './../dns';
 import {
   setResolver,
+  isDescriptionValid,
   resolver,
   addWebsiteToBlackList,
   addWebsitesToBlackList,
@@ -49,6 +50,19 @@ describe('DNS contract tests', () => {
     expect(Storage.get(contractOwnerKey)).toStrictEqual(serializedDnsAdmin);
   });
 
+  test('description length', () => {
+    const validDescriptionLessThan280 =
+      'Valid description with less than 280 characters.';
+    const invalidDescriptionMoreThan280 =
+      'Invalid description exceeding the maximum limit of 280 characters.' +
+      'x'.repeat(300);
+    const validDescriptionExactly280 = 'x'.repeat(280);
+
+    expect(isDescriptionValid(validDescriptionLessThan280)).toBe(true);
+    expect(isDescriptionValid(invalidDescriptionMoreThan280)).toBe(false);
+    expect(isDescriptionValid(validDescriptionExactly280)).toBe(true);
+  });
+
   test('invalid dns entry', () => {
     expect(() => {
       const setResolverArgs = new Args()

smart-contracts/assembly/contracts/dns/dns.ts

@@ -55,6 +55,15 @@ function isDnsValid(input: string): bool {
   return true;
 }
 
+export function isDescriptionValid(description: string): boolean {
+  // Check if the length exceeds the maximum limit of 280 characters
+  if (description.length > 280) {
+    return false;
+  }
+
+  return true;
+}
+
 /**
  * This function is meant to be called only one time: when the contract is deployed.
  *
@@ -122,6 +131,10 @@ export function setResolver(binaryArgs: StaticArray<u8>): void {
     .nextString()
     .expect('website description is missing or invalid');
 
+  if (!isDescriptionValid(description)) {
+    triggerError('INVALID_DESCRIPTION_ENTRY');
+  }
+
   if (Storage.has(websiteNameBytes)) {
     triggerError('Try another website name, this one is already taken.');
   }