@johntalton/http-util 6.1.0 → 7.0.2

package/src/headers/range.js

@@ -53,8 +53,8 @@ export class Range {
 	static parse(rangeHeader) {
 		if(rangeHeader === undefined) { return undefined }
 		if(!rangeHeader.startsWith(RANGE_UNITS_BYTES)) { return undefined }
-		if(!(rangeHeader.substring(RANGE_UNITS_BYTES.length, RANGE_UNITS_BYTES.length + 1) === RANGE_EQUAL)) { return undefined }
-		const rangeStr = rangeHeader.substring(RANGE_UNITS_BYTES.length + RANGE_EQUAL.length).trim()
+		if(!(rangeHeader.slice(RANGE_UNITS_BYTES.length, RANGE_UNITS_BYTES.length + 1) === RANGE_EQUAL)) { return undefined }
+		const rangeStr = rangeHeader.slice(RANGE_UNITS_BYTES.length + RANGE_EQUAL.length).trim()
 		if(rangeStr === '') { return undefined }
 
 		const ranges = rangeStr.split(RANGE_LIST_SEPARATOR)
@@ -99,6 +99,8 @@ export class Range {
 	 */
 	static normalize(directive, contentLength) {
 		if(directive === undefined) { return undefined }
+		if(!Number.isInteger(contentLength)) { return undefined }
+		if(contentLength <= 0) { return undefined }
 
 		/** @type {Array<NormalizedRangeValue>} */
 		const normalizedRanges = directive.ranges.map(({ start, end }) => {

package/src/headers/rate-limit.js

@@ -1,6 +1,8 @@
+import { COMMON_LIST_HEADER_JOINER_COMMA } from "../defs.js"
+
 // https://www.ietf.org/archive/id/draft-ietf-httpapi-ratelimit-headers-10.html
 
-export  const HTTP_HEADER_RATE_LIMIT = 'RateLimit'
+export const HTTP_HEADER_RATE_LIMIT = 'RateLimit'
 export const HTTP_HEADER_RATE_LIMIT_POLICY = 'RateLimit-Policy'
 
 /**
@@ -41,10 +43,17 @@ export const QUOTA_UNIT = {
 }
 
 export class RateLimit {
+	/**
+	 * @deprecated
+	 * @see {@link RateLimit.encode}
+	 * @param {RateLimitInfo} limitInfo
+	 */
+	static from(limitInfo) { return RateLimit.encode(limitInfo) }
+
 	/**
 	 * @param {RateLimitInfo} limitInfo
 	 */
-	static from(limitInfo) {
+	static encode(limitInfo) {
 		if(limitInfo === undefined) { return undefined }
 		const { name, remaining, resetSeconds, partitionKey } = limitInfo
 
@@ -57,10 +66,17 @@ export class RateLimit {
 }
 
 export class RateLimitPolicy {
+	/**
+	 * @deprecated
+	 * @see {@link RateLimitPolicy.encode}
+	 * @param {...RateLimitPolicyInfo} policies
+	 */
+	static from(...policies) { return RateLimitPolicy.encode(...policies) }
+
 	/**
 	 * @param {...RateLimitPolicyInfo} policies
 	 */
-	static from(...policies) {
+	static encode(...policies) { // todo AsArray
 		if(policies === undefined) { return undefined }
 		if(policies.length === 0) { return undefined }
 
@@ -86,6 +102,6 @@ export class RateLimitPolicy {
 					.filter(item => item !== undefined)
 					.join(';')
 			})
-			.join(',')
+			.join(COMMON_LIST_HEADER_JOINER_COMMA)
 	}
 }

package/src/headers/server-timing.js

@@ -20,12 +20,13 @@ export const SERVER_TIMING_SEPARATOR = {
 export class ServerTiming {
 	/**
 	 * @param {Array<TimingsInfo>|undefined} timings
+	 * @param {boolean} [asArray = false]
 	 */
-	static encode(timings) {
+	static encode(timings, asArray = false) {
 		if(timings === undefined) { return undefined }
 		if(timings.length <= 0) { return undefined }
 
-		return timings
+		const ary = timings
 			.map(({ name, duration, description }) => [
 					`${name}`,
 					description === undefined ? undefined : `${SERVER_TIMING_KEY_DESCRIPTION}${SERVER_TIMING_SEPARATOR.KVP}"${description}"`,
@@ -33,6 +34,7 @@ export class ServerTiming {
 				]
 				.filter(item => item !== undefined)
 				.join(SERVER_TIMING_SEPARATOR.PARAMETER))
-			.join(SERVER_TIMING_SEPARATOR.METRIC)
+
+		return asArray ? ary : ary.join(SERVER_TIMING_SEPARATOR.METRIC) // todo COMMON_LIST_HEADER_JOINER_COMMA
 	}
 }

package/src/headers/util/kvp.js

@@ -5,7 +5,6 @@ export const DEFAULT_DELIMITER = ';'
 export const KVP_DELIMITER = '='
 export const KVP_EMPTY = ''
 
-
 export class KVP {
 	/**
 	 * @param {Array<string>|undefined} params
@@ -40,6 +39,7 @@ export class KVP {
 	}
 
 	/**
+	 * Parses Key Value pair parameter list with leading named item.
 	 * @param {string|undefined} str
 	 * @param {Array<string>|undefined} [acceptableKeys=undefined]
 	 */
@@ -58,6 +58,7 @@ export class KVP {
 	}
 
 	/**
+	 * Parses Key Value pair parameter list (no special treatment for leading key)
 	 * @param {string|undefined} str
 	 * @param {Array<string>|undefined} [acceptableKeys=undefined]
 	 */

package/src/headers/util/mime.js

@@ -19,7 +19,7 @@ export const SPECIAL_CHARS = [
 	'{', '}',
 	'@', ',', ';', ':',
 	'\\', '"', '/', '?', '=', // '.',
-	// '%', // '!', '$', '&', // #  ^ * | ~ `
+	// '%', // '!', '$', '&', // # ^ * | ~ `
 	// space
 	' ', '\u000B', '\u000C',
 	// control
@@ -74,4 +74,20 @@ export class Mime {
 			subtype
 		}
 	}
+
+	/**
+	 * @param {MimeItem} first
+	 * @param {MimeItem} second
+	 * @returns {boolean}
+	 */
+	static matches(first, second) {
+		if(first === undefined) { return false }
+		if(second === undefined) { return false }
+
+		const matchType = first.type === second.type || first.type === MIME_ANY || second.type === MIME_ANY
+		if(!matchType) { return false }
+
+		const matchSubtype = first.subtype === second.subtype || first.subtype === MIME_ANY || second.subtype === MIME_ANY
+		return matchSubtype
+	}
 }

package/src/headers/util/quote.js

@@ -5,7 +5,7 @@ export const QUOTE = '"'
  */
 export function stripQuotes(value) {
 	if(value === undefined) { return undefined }
-	return value.substring(1, value.length - 1)
+	return value.slice(1, -1)
 }
 
 /**

package/src/headers/util/whitespace.js

@@ -4,5 +4,5 @@ export const WHITESPACE_REGEX = /\s/
  * @param {string} c
  */
 export function isWhitespace(c){
-  return WHITESPACE_REGEX.test(c)
+	return WHITESPACE_REGEX.test(c)
 }

package/src/headers/www-authenticate.js

@@ -1,3 +1,5 @@
+import { COMMON_LIST_HEADER_JOINER_COMMA, COMMON_LIST_VALUE_JOINER_COMMA } from "../defs.js"
+
 /**
  * @typedef {Object} ChallengeItem
  * @property {string} scheme
@@ -31,14 +33,15 @@ export function paramNeedQuotes(paramName) {
 	return PARAMETERS_THAT_NEED_QUOTES.includes(paramName.toLowerCase())
 }
 
-
 export class Challenge {
 	/**
 	 * @param {string} realm
 	 * @param {string} [charset='utf-8']
-	 * @returns {ChallengeItem}
+	 * @returns {ChallengeItem|undefined}
 	 */
 	static basic(realm, charset = 'utf-8') {
+		if(realm === undefined) { return undefined }
+
 		const parameters = new Map([ [ 'realm', realm ] ])
 		if(charset !== undefined) { parameters.set('charset', charset) }
 
@@ -53,10 +56,10 @@ export class Challenge {
 	 * @param {string|undefined} [scope]
 	 * @param {BearerErrorCode} [error]
 	 * @param {string} [errorDescription]
-	 * @param {string} [errorUri]
+	 * @param {string} [_errorUri]
 	 * @returns {ChallengeItem}
 	 */
-	static bearer(realm, scope, error, errorDescription, errorUri) {
+	static bearer(realm, scope, error, errorDescription, _errorUri) {
 		const parameters = new Map()
 		if(realm !== undefined) { parameters.set('realm', realm) }
 		if(scope !== undefined) { parameters.set('scope', scope) }
@@ -70,11 +73,11 @@ export class Challenge {
 	}
 
 	/**
-	 * @param {string} algorithm
-	 * @param {string} [realm]
+	 * @param {string} _algorithm
+	 * @param {string} [_realm]
 	 * @returns {ChallengeItem}
 	 */
-	static digest(algorithm, realm) {
+	static digest(_algorithm, _realm) {
 		return {
 			scheme: 'Digest'
 		}
@@ -90,18 +93,39 @@ export class Challenge {
 	}
 
 	/**
-	 * @param {ChallengeItem} challenge
+	 * @param {Array<ChallengeItem> | ChallengeItem | undefined} challenges
+	 * @param {boolean} [asArray = false]
 	 */
-	static encode(challenge) {
+	static encode(challenges, asArray = false) {
+		if(!Array.isArray(challenges)) { return Challenge.#encode(challenges) }
+
+		const ary = challenges
+			.map(Challenge.#encode)
+			.filter(c => c !== undefined)
+
+		return asArray ? ary : ary.join(COMMON_LIST_HEADER_JOINER_COMMA)
+	}
+
+	/**
+	 * @param {ChallengeItem | undefined} challenge
+	 */
+	static #encode(challenge) {
+		if(challenge === undefined) { return undefined }
+		if(challenge.scheme === undefined) { return undefined }
+		if(challenge.scheme === '') { return undefined }
+
 		const parameters = challenge.parameters?.entries().map(([ key, value ]) => {
 			if(value === undefined) { return key }
 			if(paramNeedQuotes(key)) { return `${key}="${value}"` }
 			return `${key}=${value}`
 		})
 
-		const params = parameters === undefined ? '' : [ ...parameters ].join(',')
+		if(parameters === undefined) { return challenge.scheme }
+		const params = [ ...parameters ]
+		if(params.length === 0) { return challenge.scheme }
 
-		return `${challenge.scheme} ${params}`
+		const paramsStr = params.join(COMMON_LIST_VALUE_JOINER_COMMA)
+		return [ challenge.scheme, paramsStr ].join(' ')
 	}
 }
 

package/src/response/2xx/accepted.js

@@ -1,6 +1,6 @@
 import http2 from 'node:http2'
 
-import { send } from '../send-util.js'
+import { send_no_body } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
 /** @import { Metadata } from '../../defs.js' */
@@ -12,5 +12,5 @@ const { HTTP_STATUS_ACCEPTED } = http2.constants
  * @param {Metadata} meta
  */
 export function sendAccepted(stream, meta) {
-	send(stream, HTTP_STATUS_ACCEPTED, {}, [], undefined, undefined, meta)
+	send_no_body(stream, HTTP_STATUS_ACCEPTED, {}, [], meta)
 }

package/src/response/2xx/bytes.js

@@ -3,39 +3,10 @@ import http2 from 'node:http2'
 import { send_bytes } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
-/** @import { AcceptRangeUnits, SendContent, SendInfo, Metadata, SendBody } from '../../defs.js' */
-/** @import { EtagItem, IMFFixDateInput } from '../../headers/conditional.js' */
-/** @import { CacheControlOptions } from '../../headers/cache-control.js' */
+/** @import { SendContent, SendInfo, Metadata, SendBody } from '../../defs.js' */
 
 const { HTTP_STATUS_OK } = http2.constants
 
-/**
- * @param {ServerHttp2Stream} stream
- * @param {SendBody|undefined} obj
- * @param {string|undefined} contentType
- * @param {number|undefined} contentLength
- * @param {string|undefined} encoding
- * @param {EtagItem|undefined} etag
- * @param {IMFFixDateInput|string|undefined} lastModified
- * @param {number|undefined} age
- * @param {CacheControlOptions} cacheControl
- * @param {AcceptRangeUnits|undefined} acceptRanges
- * @param {Metadata} meta
- */
-export function sendBytes(stream, contentType, obj, contentLength, encoding, etag, lastModified, age, cacheControl, acceptRanges, meta) {
-	_sendBytes(stream, obj, {
-		contentType,
-		contentLength,
-		encoding,
-		etag,
-		lastModified,
-		age,
-		cacheControl
-	}, {
-		acceptRanges
-	}, meta)
-}
-
 /**
  * @param {ServerHttp2Stream} stream
  * @param {SendBody|undefined} obj
@@ -43,7 +14,7 @@ export function sendBytes(stream, contentType, obj, contentLength, encoding, eta
  * @param {Pick<SendInfo, 'acceptRanges'>} info
  * @param {Metadata} meta
  */
-export function _sendBytes(stream, obj, content, info, meta) {
+export function sendBytes(stream, obj, content, info, meta) {
 	const {
 		contentType,
 		contentLength,

package/src/response/2xx/created.js

@@ -1,11 +1,10 @@
 import http2 from 'node:http2'
 
 import { Conditional } from '../../headers/conditional.js'
-import { send } from '../send-util.js'
+import { send_no_body } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
 /** @import { SendContent, Metadata } from '../../defs.js' */
-/** @import { EtagItem, IMFFixDateInput } from '../../headers/conditional.js' */
 
 const {
 	HTTP2_HEADER_LOCATION,
@@ -17,33 +16,21 @@ const { HTTP_STATUS_CREATED } = http2.constants
 
 /**
  * @param {ServerHttp2Stream} stream
- * @param {URL} location
- * @param {EtagItem|undefined} etag
- * @param {IMFFixDateInput|string|undefined} lastModified
- * @param {Metadata} meta
- */
-export function sendCreated(stream, location, etag, lastModified, meta) {
-	_sendCreated(stream, location, {
-		etag,
-		lastModified
-	}, meta)
-}
-
-/**
- * @param {ServerHttp2Stream} stream
- * @param {URL} location
+ * @param {URL|string} location
  * @param {Pick<SendContent, 'etag' | 'lastModified'>} content
  * @param {Metadata} meta
  */
-export function _sendCreated(stream, location, content, meta) {
+export function sendCreated(stream, location, content, meta) {
 	const {
 		etag,
 		lastModified
 	} = content
 
-	send(stream, HTTP_STATUS_CREATED, {
-			[HTTP2_HEADER_LOCATION]: location.href,
+	const loc = (location instanceof URL) ? location.href : location
+
+	send_no_body(stream, HTTP_STATUS_CREATED, {
+			[HTTP2_HEADER_LOCATION]: loc,
 			[HTTP2_HEADER_ETAG]: Conditional.encodeEtag(etag),
 			[HTTP2_HEADER_LAST_MODIFIED]: Conditional.encodeFixDate(lastModified)
-		}, [ HTTP2_HEADER_LOCATION ], undefined, undefined, meta)
+		}, [ HTTP2_HEADER_LOCATION ], meta)
 }

package/src/response/2xx/json.js

@@ -5,32 +5,19 @@ import { send_encoded } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
 /** @import { SendContent, SendInfo, Metadata } from '../../defs.js' */
-/** @import { EtagItem, IMFFixDateInput } from '../../headers/conditional.js' */
-/** @import { CacheControlOptions } from '../../headers/cache-control.js' */
 
 const { HTTP_STATUS_OK } = http2.constants
 
 /**
+ * @deprecated
  * @param {ServerHttp2Stream} stream
  * @param {Object} obj
- * @param {string|undefined} encoding
- * @param {EtagItem|undefined} etag
- * @param {IMFFixDateInput|string|undefined} lastModified
- * @param {number|undefined} age
- * @param {CacheControlOptions} cacheControl
- * @param {Array<string>|undefined} supportedQueryTypes
+ * @param {Omit<SendContent, 'contentType' | 'contentLength' | 'rangeDirective'>} content
+ * @param {Pick<SendInfo, 'supportedQueryTypes'>} info
  * @param {Metadata} meta
  */
-export function sendJSON_Encoded(stream, obj, encoding, etag, lastModified, age, cacheControl, supportedQueryTypes, meta) {
-	_sendJSON_Encoded(stream, obj, {
-		encoding,
-		etag,
-		lastModified,
-		age,
-		cacheControl
-	}, {
-		supportedQueryTypes
-	}, meta)
+export function sendJSON_Encoded(stream, obj, content, info, meta) {
+	sendJSON(stream, obj, content, info, meta)
 }
 
 /**
@@ -40,7 +27,7 @@ export function sendJSON_Encoded(stream, obj, encoding, etag, lastModified, age,
  * @param {Pick<SendInfo, 'supportedQueryTypes'>} info
  * @param {Metadata} meta
  */
-export function _sendJSON_Encoded(stream, obj, content, info, meta) {
+export function sendJSON(stream, obj, content, info, meta) {
 	const {
 		encoding,
 		etag,

package/src/response/2xx/no-content.js

@@ -1,11 +1,10 @@
 import http2 from 'node:http2'
 
 import { Conditional } from '../../headers/conditional.js'
-import { send } from '../send-util.js'
+import { send_no_body } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
 /** @import { SendContent, Metadata } from '../../defs.js' */
-/** @import { EtagItem, IMFFixDateInput } from '../../headers/conditional.js' */
 
 const {
 	HTTP2_HEADER_ETAG,
@@ -14,32 +13,19 @@ const {
 
 const { HTTP_STATUS_NO_CONTENT } = http2.constants
 
-/**
- * @param {ServerHttp2Stream} stream
- * @param {EtagItem|undefined} etag
- * @param {IMFFixDateInput|string|undefined} lastModified
- * @param {Metadata} meta
- */
-export function sendNoContent(stream, etag, lastModified, meta) {
-	_sendNoContent(stream, {
-		etag,
-		lastModified
-	}, meta)
-}
-
 /**
  * @param {ServerHttp2Stream} stream
  * @param {Pick<SendContent, 'etag' | 'lastModified'>} content
  * @param {Metadata} meta
  */
-export function _sendNoContent(stream, content, meta) {
+export function sendNoContent(stream, content, meta) {
 	const {
 		etag,
 		lastModified
 	} = content
 
-	send(stream, HTTP_STATUS_NO_CONTENT, {
+	send_no_body(stream, HTTP_STATUS_NO_CONTENT, {
 			[HTTP2_HEADER_ETAG]: Conditional.encodeEtag(etag),
 			[HTTP2_HEADER_LAST_MODIFIED]: Conditional.encodeFixDate(lastModified)
-		}, [], undefined, undefined, meta)
+		}, [], meta)
 }

package/src/response/2xx/partial-content.js

@@ -7,11 +7,8 @@ import { send_bytes } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
 /** @import { SendContent, Metadata, SendBody } from '../../defs.js' */
-/** @import { EtagItem, IMFFixDateInput } from '../../headers/conditional.js' */
-/** @import { CacheControlOptions } from '../../headers/cache-control.js' */
 /** @import { ContentRangeDirective } from '../../headers/content-range.js' */
 
-
 const { HTTP_STATUS_PARTIAL_CONTENT } = http2.constants
 
 /**
@@ -25,37 +22,13 @@ const { HTTP_STATUS_PARTIAL_CONTENT } = http2.constants
  * @property {ContentRangeDirective} range
  */
 
-/**
- * @param {ServerHttp2Stream} stream
- * @param {string|undefined} contentType
- * @param {NonEmptyArray<PartialBytes>|PartialBytes} objs
- * @param {number|undefined} contentLength
- * @param {string|undefined} encoding
- * @param {EtagItem|undefined} etag
- * @param {IMFFixDateInput|string|undefined} lastModified
- * @param {number|undefined} age
- * @param {CacheControlOptions} cacheControl
- * @param {Metadata} meta
- */
-export function sendPartialContent(stream, contentType, objs, contentLength, encoding, etag, lastModified, age, cacheControl, meta) {
-	return _sendPartialContent(stream, objs, {
-		contentType,
-		contentLength,
-		encoding,
-		etag,
-		lastModified,
-		age,
-		cacheControl
-	}, meta)
-}
-
 /**
  * @param {ServerHttp2Stream} stream
  * @param {NonEmptyArray<PartialBytes>|PartialBytes} objs
  * @param {Omit<SendContent, 'rangeDirective'>} content
  * @param {Metadata} meta
  */
-export function _sendPartialContent(stream, objs, content, meta) {
+export function sendPartialContent(stream, objs, content, meta) {
 	const {
 		contentType,
 		contentLength,

package/src/response/2xx/preflight.js

@@ -1,15 +1,18 @@
 import http2 from 'node:http2'
 
 import {
+	COMMON_LIST_VALUE_JOINER_COMMA,
+	// HTTP_HEADER_ACCEPT_PATCH,
+	// HTTP_HEADER_ACCEPT_POST,
 	HTTP_HEADER_ACCEPT_QUERY,
 	HTTP_METHOD_QUERY,
 	HTTP2_HEADER_ACCESS_CONTROL_MAX_AGE,
 	PREFLIGHT_AGE_SECONDS
 } from '../../defs.js'
-import { send } from '../send-util.js'
+import { send_no_body } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
-/** @import { AcceptRangeUnits, SendInfo, Metadata } from '../../defs.js' */
+/** @import { SendInfo, Metadata } from '../../defs.js' */
 
 const {
 	HTTP2_HEADER_CONTENT_TYPE,
@@ -25,27 +28,12 @@ const {
 
 const { HTTP_STATUS_OK } = http2.constants
 
-/**
- * @param {ServerHttp2Stream} stream
- * @param {Array<string>} supportedMethods
- * @param {Array<string>|undefined} supportedQueryTypes
- * @param {AcceptRangeUnits|undefined} acceptRanges
- * @param {Metadata} meta
- */
-export function sendPreflight(stream, supportedMethods, supportedQueryTypes, acceptRanges, meta) {
-	_sendPreflight(stream, {
-		supportedMethods,
-		supportedQueryTypes,
-		acceptRanges
-	}, meta)
-}
-
 /**
  * @param {ServerHttp2Stream} stream
  * @param {Pick<SendInfo, 'supportedMethods' | 'supportedQueryTypes' | 'acceptRanges'>} info
  * @param {Metadata} meta
  */
-export function _sendPreflight(stream, info, meta) {
+export function sendPreflight(stream, info, meta) {
 	const {
 		supportedMethods,
 		supportedQueryTypes,
@@ -56,19 +44,24 @@ export function _sendPreflight(stream, info, meta) {
 	const exposedHeadersAcceptQuery = supportsQuery ? [ HTTP_HEADER_ACCEPT_QUERY ] : []
 	const exposedHeaders = acceptRanges === undefined ? exposedHeadersAcceptQuery : [ HTTP2_HEADER_ACCEPT_RANGES, ...exposedHeadersAcceptQuery ]
 
-	send(stream, HTTP_STATUS_OK, {
-			[HTTP2_HEADER_ACCESS_CONTROL_ALLOW_METHODS]: supportedMethods.join(','),
+	// todo: if supportedMethods includes POST | PATCH
+	// include accept-post / accept-patch headers
+
+	send_no_body(stream, HTTP_STATUS_OK, {
+			[HTTP2_HEADER_ACCESS_CONTROL_ALLOW_METHODS]: supportedMethods.join(COMMON_LIST_VALUE_JOINER_COMMA),
 			[HTTP2_HEADER_ACCESS_CONTROL_ALLOW_HEADERS]: [
 				HTTP2_HEADER_IF_MATCH,
 				HTTP2_HEADER_IF_NONE_MATCH,
 				HTTP2_HEADER_AUTHORIZATION,
-				HTTP2_HEADER_CONTENT_TYPE,
-				HTTP2_HEADER_RANGE,
+				HTTP2_HEADER_CONTENT_TYPE, // overrides cors safe restriction (for json)
+				HTTP2_HEADER_RANGE, // todo cors safe override not needed
 				HTTP2_HEADER_IF_RANGE
-			].join(','),
+			].join(COMMON_LIST_VALUE_JOINER_COMMA),
 			[HTTP2_HEADER_ACCESS_CONTROL_MAX_AGE]: PREFLIGHT_AGE_SECONDS,
+			// [HTTP_HEADER_ACCEPT_POST]: ,
+			// [HTTP_HEADER_ACCEPT_PATCH]: ,
 			[HTTP2_HEADER_ACCEPT_RANGES]: acceptRanges,
-			[HTTP_HEADER_ACCEPT_QUERY]: supportedQueryTypes?.join(',') // todo should empty array return undef
+			[HTTP_HEADER_ACCEPT_QUERY]: supportedQueryTypes?.join(COMMON_LIST_VALUE_JOINER_COMMA) // todo should empty array return undef
 			// Access-Control-Allow-Credentials
-		}, exposedHeaders, undefined, undefined, meta)
+		}, exposedHeaders, meta)
 }

package/src/response/3xx/found.js

@@ -1,6 +1,6 @@
-import  http2 from 'node:http2'
+import http2 from 'node:http2'
 
-import { send } from '../send-util.js'
+import { send_no_body } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
 /** @import { Metadata } from '../../defs.js' */
@@ -13,11 +13,13 @@ const { HTTP_STATUS_FOUND } = http2.constants
 
 /**
  * @param {ServerHttp2Stream} stream
- * @param {URL} location
+ * @param {URL|string} location
  * @param {Metadata} meta
  */
 export function sendFound(stream, location, meta) {
-	send(stream, HTTP_STATUS_FOUND, {
-		[HTTP2_HEADER_LOCATION]: location.href
-	}, [ HTTP2_HEADER_LOCATION ], undefined, undefined, meta)
+	const loc = (location instanceof URL) ? location.href : location
+
+	send_no_body(stream, HTTP_STATUS_FOUND, {
+		[HTTP2_HEADER_LOCATION]: loc
+	}, [ HTTP2_HEADER_LOCATION ], meta)
 }

package/src/response/3xx/moved-permanently.js

@@ -1,23 +1,25 @@
 import http2 from 'node:http2'
 
-import { send } from '../send-util.js'
+import { send_no_body } from '../send-util.js'
 
 /** @import { ServerHttp2Stream } from 'node:http2' */
 /** @import { Metadata } from '../../defs.js' */
 
 const {
-  HTTP2_HEADER_LOCATION
+	HTTP2_HEADER_LOCATION
 } = http2.constants
 
 const { HTTP_STATUS_MOVED_PERMANENTLY } = http2.constants
 
 /**
  * @param {ServerHttp2Stream} stream
- * @param {URL} location
+ * @param {URL|string} location
  * @param {Metadata} meta
  */
 export function sendMovedPermanently(stream, location, meta) {
-  send(stream, HTTP_STATUS_MOVED_PERMANENTLY, {
-    [HTTP2_HEADER_LOCATION]: location.href
-  }, [ HTTP2_HEADER_LOCATION ], undefined, undefined, meta)
+	const loc = (location instanceof URL) ? location.href : location
+
+	send_no_body(stream, HTTP_STATUS_MOVED_PERMANENTLY, {
+		[HTTP2_HEADER_LOCATION]: loc
+	}, [HTTP2_HEADER_LOCATION], meta)
 }