mirror of
https://github.com/advplyr/audiobookshelf.git
synced 2026-08-29 14:47:20 +02:00
[PR #4885] Implement experimental DNS pre-resolution, enabling Podcast downloads even with partial DNS resolution failures #4365
Open
opened 2026-04-25 00:19:26 +02:00 by adam
·
0 comments
No Branch/Tag Specified
master
auth_sessions_enhancements
account_sessions_table
logout_all_devices
pw_change_invalidates_sessions
book_tags_genres_dedupe
episode_download_fallback
Issue-4540-SortBy-StartedDate-and-FinishedDate
episode_meta_tagging
fix_authorize_race_condition
redirect_transcode_requests
progress_updated_sort
fix_ereader_socket_event
fix_change_empty_root_password
fix_podcast_session_track_index
fix_set_token
session_modal_user
localize_durations
fix_oidc_create_user
jwt_auth_refactor
fix_scanner_deleting_single_file_books
fix_mediaprogress_updatedat_2
experimental_next_client
podcast_episode_duration
episode-timestamps-clickable
book_author_secondary_sort_title
podcast_useragents
pathexists_user_access
fix_pathexists_join
book_author_secondary_sort
clean_duplicate_mediaprogress
sanitize_html_description
trix_prevent_attachments
check_path_api_fix
fix_mediaprogress_updatedat
increase_express_json_limit
fix_dockerfile_nunicode
search_episodes
audiobook_tools_update
episode_secondary_sorts
hls_stream_url_update
new_session_track_endpoint
audiobook_tools_enhancements
watcher_rescans_update
player_track_tooltip
fix_exclude_prefixes_crash
socket_item_events
fix_podcast_episode_scanner_promise
new_stats_controller
count_cache_for_userpermissions
parsing-opf-v3
validate_migration_files
fix-quick-match-all-crash
fix-chapter-end-sleep-timer
stringify_sequelize_query
remove-col-ambiguity
fix_next_prev_edit_description
details_trim_whitespace
fix_content_url_basepath
fix_logger_fatal
progress_bar_visibility
batch-edit-populate-map-details
feed_generator_updates
bookmark-modal-updates
migrate-library-item-in-scanner
migrate-new-library-items
migrate-podcasts-new-library-item-2
migrate-podcasts-new-library-item
fix-remove-episode-from-playlist
playback-session-use-new-library-item
refactor-library-item
fix-heatmap-caption
feed-episodes-upsert
share-media-player-media-session-api
remove-old-playlist
remove_old_collection_object
plugin-implementation-demo
feed_migration
refactor-feeds-from-item
fix_remove_authors_no_books
v2.17.3-fk-constraints-migration
migrations-first-upgrade
sqlite_2
feature/nuxt-target-server
waveform
sqlite
playlists
video
v2.36.0
v2.35.1
v2.35.0
v2.34.0
v2.33.2
v2.33.1
v2.33.0
v2.32.1
v2.32.0
v2.31.0
v2.30.0
v2.29.0
v2.28.0
v2.27.0
v2.26.3
v2.26.2
v2.26.1
v2.26.0
v2.25.1
v2.25.0
v2.24.0
v2.23.0
v2.22.0
v2.21.0
v2.20.0
v2.19.5
v2.19.4
v2.19.3
v2.19.2
v2.19.1
v2.19.0
v2.18.1
v2.18.0
v2.17.7
v2.17.6
v2.17.5
v2.17.4
v2.17.3
v2.17.2
v2.17.1
v2.17.0
v2.16.2
v2.16.1
v2.16.0
v2.15.1
v2.15.0
v2.14.0
v2.13.4
v2.13.3
v2.13.2
v2.13.1
v2.13.0
v2.12.3
v2.12.2
v2.12.1
v2.12.0
v2.11.0
v2.10.1
v2.10.0
v2.9.0
v2.8.1
v2.8.0
v2.7.2
v2.7.1
v2.7.0
v2.6.0
v2.5.0
v2.4.4
v2.4.3
v2.4.2
v2.4.1
v2.4.0
v2.3.5
v2.3.4
v2.3.3
v2.3.2
v2.3.1
v2.3.0
v2.2.23
v2.2.22
v2.2.21
v2.2.20
v2.2.19
v2.2.18
v2.2.17
v2.2.16
v2.2.15
v2.2.14
v2.2.13
v2.2.12
v2.2.11
v2.2.10
v2.2.9
v2.2.8
v2.2.7
v2.2.6
v2.2.5
v2.2.4
v2.2.3
v2.2.2
v2.2.1
v2.2.0
v2.1.5
v2.1.4
v2.1.3
v2.1.2
v2.1.1
v2.1.0
v2.0.24
v2.0.23
v2.0.22
v2.0.21
v2.0.20
v2.0.19
v2.0.18
v2.0.17
v2.0.16
v2.0.15
v2.0.14
v2.0.13
v2.0.12
v2.0.11
v2.0.10
v2.0.9
v2.0.8
v2.0.7
v2.0.6
v2.0.5
v2.0.4
v2.0.3
v2.0.2
v2.0.1
v1.7.2
v1.7.1
v1.7.0
v1.6.0
v1.5.5
v1.5.0
v1.4.11
v1.4.9
v1.4.7
v1.4.6
v1.4.4
v1.4.2
v1.4.0
v1.4.1
v1.3.4
v1.3.3
v1.3.1
v1.2.8
v1.2.6
v1.2.5
v1.2.4
v1.2.1
v1.1.15
v1.1.14
v1.1.13
v1.1.12
v1.1.11
v1.1.10
v1.1.9
v1.1.8
v1.0.0
0.9.61-beta.0
0.9.61-beta
Labels
Clear labels
authentication
backlog
bug
chapter editor
config-issue
ebooks
encoding/embedding
enhancement
help wanted
listening sessions & progress
planned
possible plugin
progress sync
pull-request
sorting/filtering/searching
unable to reproduce
upload
users & permissions
waiting
Mirrored from GitHub Pull Request
No labels
pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
adam (Adam Melkus)
Clear assignees
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/audiobookshelf#4365
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/advplyr/audiobookshelf/pull/4885
Author: @PassionateBytes
Created: 12/2/2025
Status: 🔄 Open
Base:
master← Head:fix-dns-resolution📝 Commits (3)
da040c7Implement experimental DNS pre-resolution33d2839Use experimental DNS resolution on redirectsfc81be0Handle redirect loops and maximum redirect limits📊 Changes
1 file changed (+78 additions, -0 deletions)
View changed files
📝
server/Server.js(+78 -0)📄 Description
Brief summary
This PR introduces an optional configuration parameter
EXP_DNS_RESOLUTIONto the server, which enables a custom mechanism for pre-resolving hostnames in allaxiosweb requests. This mechanism uses the dedicateddnspackage to explicitly resolveAandAAAArecords. This bypasses thegetaddrinfosyscall, which in the Alpine container environment fails entirely, if only part of the DNS query results in aSERVFAIL.This enables successful web requests in situations that would otherwise fail due to partial DNS resolution failures. - i.e. situations in which a DNS query resolves an
Arecord of a hostname, while rejecting requests forAAAArecords (R-CodeSERVFAIL, orREFUSED). I have found that this causes thegetaddrinfosyscall to fail entirely inside the Alpine container environment specifically, with the result that the entire web request fails. (Note that this is an alpine specific problem).Enabling this new feature with
EXP_DNS_RESOLUTION=1will trigger explicit DNS resolution, with graceful fallback between IPv4 and IPv6. If either fails to resolve, while the other succeeds, the web request will still be allowed to go through, rather than failing entirely.Which issue is fixed?
Additionally, a discussion of the issue server side can be found here:
...potentially more than just these issues - I have seen the same problem manifest for multiple different podcasts myself, so I expect to find more related issues. Will add them here when I come across more.
In-depth Description
It appears that in some environments axios' fails to execute a web request entirely, when a domain name gets resolved only partialy.
This can be the case when a DNS server is misconfigured, or overly restrictive, such that it only resolves IPv4 addresses (
Arecords) successfully, while rejecting IPv6 address requests (AAAArecords) withSERVFAIL(or vise versa).Consider the example below:
Understanding Status Quo:
As an example, the German News Podcast 'Lage der Nation' hosts their public feeds on
feeds.lagedernation.organd their episodes oncdn.lagedernation.org.Resolving these domains returns IPv4 addresses (
Arecord) successfully. However, their DNS server configuration appears to be faulty, because it consistently leads to aSERVFAILwhen attempting to resolve for IPv6 addresses (AAAArecords).This failure mode can be replicated in various environments:
In certain environments, axios appears to fail the entire web request when it encounters such a partial DNS resolution failure, even though a valid IPv4 address was returned. This results in errors like:
Full Error Message...
A discussion of this particular issue can be found here
A known workaround is to disable IPv6 entirely, but that is not generally feasible for everyone and not future proof.
The Solution:
This PR implements a custom DNS resolution mechanism that bypasses axios' default DNS handling (which relies on the OS's
getaddrinfo()system call and can fail in certain environments).My implementation hooks into the global axios client, following existing similar patterns in the codebase (analogous to the
EXP_PROXY_SUPPORTfeature).When enabled via
EXP_DNS_RESOLUTION=1, the following changes are applied:1. Explicit DNS resolution before executing web requests
A request interceptor is added to the global axios instance that:
Arecords viadns.resolve4()) and IPv6 (AAAArecords viadns.resolve6()) addressesPREFER_IPV6=1flag to reverse the priorityAandAAAAseparately, allowing for independent, graceful handling of failuresHostheader of the request2. Explicit redirect handling
Axios' built-in redirect following feature would bypass the custom DNS resolution interceptor. Thus automatic redirect handling is disabled (
maxRedirects = 0) and a response interceptor is added that:The Result:
With this implementation, web requests to domains with partial DNS resolution failures will succeed when
EXP_DNS_RESOLUTION=1is set, as the custom logic ensures that at least one valid IP address is used for the request.How have you tested this?
Tested against the paid
feeds.lagedernation.organdcdn.lagedernation.orgwhich both exhibits the partial DNS resolution failure:EXP_DNS_RESOLUTION=1: axios requests fail completely, failing to load the RSS feed as well ass podcast audio filesEXP_DNS_RESOLUTION=1: requests succeed by explicitly using IPv4 resolutionEXP_DNS_RESOLUTION=1andPREFER_IPV6=1: requests succeed, even though initial IPv6 resolution fails withSERVFAIL, but it gracefully falls back to IPv4 resolutionThe implementation also handles gracefully:
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.