Initial v1 #2

2025-07-31T06:32:04Z

FrauJulian commented

2025-07-31 06:32:04 +00:00

(Migrated from github.com)

This pull request introduces a DNS performance testing script, along with supporting documentation and a domain list. The changes include the addition of the PowerShell script, a comprehensive README.md file detailing usage and functionality, and a domain list file.

New Features and Functionality:

v1.ps1: Added a PowerShell script to measure DNS query response times for domains. The script accepts parameters for DNS server, number of queries, input domain file, and output results file, and outputs response times to a CSV file along with basic statistics.

Documentation:

README.md: Added detailed documentation covering the script's purpose, parameters, installation steps, usage, output format, statistics, and an example. It also includes licensing and contributor information.

Supporting Data:

Domain Lists/100.txt: Added a list of 100 commonly used domains for use with the DNS performance testing script.

This pull request introduces a DNS performance testing script, along with supporting documentation and a domain list. The changes include the addition of the PowerShell script, a comprehensive `README.md` file detailing usage and functionality, and a domain list file. ### New Features and Functionality: * [`v1.ps1`](diffhunk://#diff-b25459ba1ec1678d50bac813ecfa87dcf4d9c9dc667f617ef6f2872e11c685a8R1-R46): Added a PowerShell script to measure DNS query response times for domains. The script accepts parameters for DNS server, number of queries, input domain file, and output results file, and outputs response times to a CSV file along with basic statistics. ### Documentation: * [`README.md`](diffhunk://#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R1-R119): Added detailed documentation covering the script's purpose, parameters, installation steps, usage, output format, statistics, and an example. It also includes licensing and contributor information. ### Supporting Data: * [`Domain Lists/100.txt`](diffhunk://#diff-46314361dbc0e4dd6310813579bfb2594d6fa4846a9bfe2c8d75ff78823e29b8R1-R100): Added a list of 100 commonly used domains for use with the DNS performance testing script.

copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed

2025-07-31 06:32:34 +00:00

copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment

Pull Request Overview

This pull request introduces a DNS performance testing tool written in PowerShell, designed to measure and analyze DNS query response times across multiple domains. The script provides configurable parameters for DNS server selection, query frequency, and file paths, outputting results in CSV format with basic statistical analysis.

Added a PowerShell script (v1.ps1) that performs DNS performance testing with configurable parameters and CSV output
Created comprehensive documentation (README.md) with usage instructions, parameter descriptions, and examples
Included domain lists for testing purposes with 100 and 1000 commonly used domains

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 2 comments.

File	Description
v1.ps1	Core PowerShell script implementing DNS performance testing functionality with error handling and statistics
README.md	Comprehensive documentation covering installation, usage, parameters, and examples
Domain Lists/100.txt	Curated list of 100 popular domains for DNS testing
Domain Lists/1000.txt	Extended list of 1000 domains for comprehensive DNS performance testing

Comments suppressed due to low confidence (2)

README.md:47

The filename 'DNSPerformanceTest.ps1' in the documentation doesn't match the actual script filename 'v1.ps1'. This inconsistency will cause confusion for users.

.\DNSPerformanceTest.ps1

README.md:88

[nitpick] The script name 'v1.ps1' is not descriptive and doesn't follow PowerShell naming conventions. Consider using a more descriptive name like 'Test-DNSPerformance.ps1' or 'DNSPerformanceTest.ps1'.

.\v1.ps1

## Pull Request Overview This pull request introduces a DNS performance testing tool written in PowerShell, designed to measure and analyze DNS query response times across multiple domains. The script provides configurable parameters for DNS server selection, query frequency, and file paths, outputting results in CSV format with basic statistical analysis. - Added a PowerShell script (`v1.ps1`) that performs DNS performance testing with configurable parameters and CSV output - Created comprehensive documentation (`README.md`) with usage instructions, parameter descriptions, and examples - Included domain lists for testing purposes with 100 and 1000 commonly used domains ### Reviewed Changes Copilot reviewed 4 out of 6 changed files in this pull request and generated 2 comments. | File | Description | | ---- | ----------- | | v1.ps1 | Core PowerShell script implementing DNS performance testing functionality with error handling and statistics | | README.md | Comprehensive documentation covering installation, usage, parameters, and examples | | Domain Lists/100.txt | Curated list of 100 popular domains for DNS testing | | Domain Lists/1000.txt | Extended list of 1000 domains for comprehensive DNS performance testing | <details> <summary>Comments suppressed due to low confidence (2)</summary> **README.md:47** * The filename 'DNSPerformanceTest.ps1' in the documentation doesn't match the actual script filename 'v1.ps1'. This inconsistency will cause confusion for users. ``` .\DNSPerformanceTest.ps1 ``` **README.md:88** * [nitpick] The script name 'v1.ps1' is not descriptive and doesn't follow PowerShell naming conventions. Consider using a more descriptive name like 'Test-DNSPerformance.ps1' or 'DNSPerformanceTest.ps1'. ``` .\v1.ps1 ``` </details>

README.md

						
				@@ -0,0 +1,111 @@

				﻿# DNS Performance Test Script

copilot-pull-request-reviewer[bot] (Migrated from github.com) commented

2025-07-31 06:32:34 +00:00

The command uses 'dotnet-script' which is incorrect for PowerShell execution. It should be '.\DNSPerformanceTest.ps1' or 'powershell DNSPerformanceTest.ps1'.

.\DNSPerformanceTest.ps1 `
    -DnsServer "1.1.1.1" `
    -QueriesPerDomain 3 `
    -DomainsFile "Domains.txt" `

The command uses 'dotnet-script' which is incorrect for PowerShell execution. It should be '.\DNSPerformanceTest.ps1' or 'powershell DNSPerformanceTest.ps1'. ```suggestion .\DNSPerformanceTest.ps1 ` -DnsServer "1.1.1.1" ` -QueriesPerDomain 3 ` -DomainsFile "Domains.txt" ` ```

v1.ps1

						
				@@ -0,0 +1,44 @@

				param(

copilot-pull-request-reviewer[bot] (Migrated from github.com) commented

2025-07-31 06:32:33 +00:00

The comment contains German text 'Domains einlesen' which should be in English for consistency. Consider changing to '# Read domains'.

# Read domains

The comment contains German text 'Domains einlesen' which should be in English for consistency. Consider changing to '# Read domains'. ```suggestion # Read domains ```

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: fraujulian/Bash-Scripts#2

				`@@ -0,0 +1,44 @@`
				`param(`