Duplication of descriptor request digest list construction code

MicrodescRequest::make_request and RouterDescRequest::make_request achieve very similar results with similar but not identical code for handling digests.

Also this code needlessly clones the digests themselves to sort them, when a vec of references could be sorted just as well.

Noticed while reviewing !553 (merged)

See also the TODO in pub struct RouterDescRequest