Skip to content

Commit

Permalink
fmt and cleanup comment
Browse files Browse the repository at this point in the history
  • Loading branch information
blaine-arcjet committed Apr 8, 2024
1 parent f422232 commit 2b50930
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
6 changes: 4 additions & 2 deletions protocol/convert.ts
Original file line number Diff line number Diff line change
Expand Up @@ -437,9 +437,11 @@ export function ArcjetIpDetailsFromProtocol(
city: ipDetails.city !== "" ? ipDetails.city : undefined,
region: ipDetails.region !== "" ? ipDetails.region : undefined,
country: ipDetails.country !== "" ? ipDetails.country : undefined,
countryName: ipDetails.countryName !== "" ? ipDetails.countryName : undefined,
countryName:
ipDetails.countryName !== "" ? ipDetails.countryName : undefined,
continent: ipDetails.continent !== "" ? ipDetails.continent : undefined,
continentName: ipDetails.continentName !== "" ? ipDetails.continentName : undefined,
continentName:
ipDetails.continentName !== "" ? ipDetails.continentName : undefined,
asn: ipDetails.asn !== "" ? ipDetails.asn : undefined,
asnName: ipDetails.asnName !== "" ? ipDetails.asnName : undefined,
asnDomain: ipDetails.asnDomain !== "" ? ipDetails.asnDomain : undefined,
Expand Down
1 change: 0 additions & 1 deletion protocol/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -439,7 +439,6 @@ export class ArcjetIpDetails {
}

// If we have ASN, we should have every piece of ASN information.
// TODO: Verify the above comment and implementation below
hasASN(): this is RequiredProps<
this,
"asn" | "asnName" | "asnDomain" | "asnType" | "asnCountry"
Expand Down

0 comments on commit 2b50930

Please sign in to comment.