CollectionSerializer is duplicating "Items" link

[ahuarte47]

Describe the bug
db_to_stac method to transform database model to STAC collection is adding a set of links by default. "Items" link is not included in the list of inferred. So the later invoking to resolve_links adds it again because "items" passes this filter.

To Reproduce
Steps to reproduce the behavior:

1. Uploads a Collection to ES
2. Reads it using CollectionSerializer:db_to_stac()

Expected behavior
"Items" link should be unique. INFERRED_LINK_RELS should include "Items" or taking care of this issue in Serializer.

[jonhealy1]

Hi @ahuarte47 do you have a fix for this?

[ahuarte47]

Hi @jonhealy1 I did create any PR to fix this, I only changed my own copy of stac-fastapi including "items" in the list INFERRED_LINK_RELS, it works here. It does not seem a bug from stac-fastapi-elasticsearch, but it in combination with stac-fastapi.

[StijnCaerts]

I created a PR in the upstream repository: stac-utils/stac-fastapi#640

[jonhealy1]

Nice ok. Thank you.

[StijnCaerts]

#211 also fixes this, as the generated links are no longer stored in the database.