control: unmerge 986356
control: merge 986356 954904
control: retitle 954904 Support SRV record
control: retitle 986356 Support SRV record
control: clone 986356 -1 -2
control: retitle -1 Add cdn-fastly.deb.debian.org to debvol_mirrors.gz
control: tags -1 patch
control: retitle -2 Truncated InRElease file
control: severity -2 normal 
thanks

Although
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=914852
is likely caused by the same root cause, let me not get too agressive.
So I am unmerging it.  But merging keep 2 bugs together.

As I grep the source for "SRV", I do not see it in apt-cacher-ng
source.  So I am faily sure apt-cacher-ng doesn't support SRV record.

https://en.wikipedia.org/wiki/SRV_record

APT is also written in C++ and it has:
  apt/apt-pkg/contrib/srvrec.cc
  apt/apt-pkg/contrib/srvrec.h
to support SRV record from 2015.

It explains as:
   DNS SRV record support in apt
   =============================
   
   Apt supports a subset of the DNS SRV server records protocol as
   described in [RFC 2782](https://tools.ietf.org/html/rfc2782) for
   service discovery.
   
   Before connecting to the requested server APT will send a SRV
   record request of the form `_$protocol._tcp._$host`, e.g.
   `_http._tcp.ftp.debian.org` or `_http._tcp.security.debian.org`.
   
   If the server sends SRV records
   as a reply APT will use those to connect to the server(s) in
   this reply. It will honor the `priority` field in the reply.
   
   However it does not implement the `weight` algorithm as described
   in RFC 2782. It will use an equal weight for each server of the
   same priority.
   
   If connecting to a server fails APT will retry with the next one
   and remove the server from the list of valid servers for this
   session.
   
Merging this part of code from apt to apt-cacher-ng should fix issues
from the root cause.  I am afraid some repos such as one from Ubuntu
may be using the similar configuration.  So other bugs may be solved
too.

Fixing this may fix truncated file problem too.  (I don't know)

I also cloned this bug to make different issues separate.

Since without adding cdn-fastly.deb.debian.org as attached file, it
waste cache space if the user uses it in chroot etc..  Since it is
small gz file I atach fixed file itselg here and mark this bug as
patch.  This goes to -1 cloned bug.

The last -2 cloned bug is for the truncated file.  Maybe this is non-
issue if this is how you designed.  But if this is caused by broken
download by the redirected service, the above may fix this.  If this is
something else, we need to keep it as separate bug.

Sorry for messing up BTS.  But I hope this is now clearler.

For now, I updated some wiki pages not to use deb.debian.org URL.

Osamu 


Attachment: deb_mirrors.gz
Description: application/gzip

Reply via email to