fix: link (#3987)

packages/service/common/system/utils.ts

@@ -30,10 +30,10 @@ export const isInternalAddress = (url: string): boolean => {
       return true;
       return true;
     }
     }
 
 
-    // For non-metadata URLs, check if it's a domain name
+    // For IP addresses, check if they are internal
     const ipv4Pattern = /^(\d{1,3}\.){3}\d{1,3}$/;
     const ipv4Pattern = /^(\d{1,3}\.){3}\d{1,3}$/;
     if (!ipv4Pattern.test(hostname)) {
     if (!ipv4Pattern.test(hostname)) {
-      return true;
+      return false; // Not an IP address, so it's a domain name - consider it external by default
     }
     }
 
 
     // ... existing IP validation code ...
     // ... existing IP validation code ...

projects/app/src/pages/api/core/dataset/collection/create/link.ts

@@ -29,8 +29,8 @@ async function handler(req: NextApiRequest): CreateCollectionResponse {
   });
   });
   const { title = link, content = '' } = result[0];
   const { title = link, content = '' } = result[0];
 
 
-  if (!content) {
-    return Promise.reject('Can not fetch content from link');
+  if (!content || content === 'Cannot fetch internal url') {
+    return Promise.reject(content || 'Can not fetch content from link');
   }
   }
 
 
   const { collectionId, insertResults } = await createCollectionAndInsertData({
   const { collectionId, insertResults } = await createCollectionAndInsertData({
@@ -38,7 +38,7 @@ async function handler(req: NextApiRequest): CreateCollectionResponse {
     rawText: content,
     rawText: content,
     createCollectionParams: {
     createCollectionParams: {
       ...body,
       ...body,
-      name: title,
+      name: title || link,
       teamId,
       teamId,
       tmbId,
       tmbId,
       type: DatasetCollectionTypeEnum.link,
       type: DatasetCollectionTypeEnum.link,