✨(ingestion) handle webhooks to archive offers

[AntoineAugusti]

📝 Description

Handle Talentsoft webhooks used to archive offers.

🏷️ Type of change

• 🐛 Bug fix
• 🎢 New feature (non-breaking change that adds functionality)
• 🥁 Breaking change (a modification or feature that would cause existing functionality to stop working as expected) requiring a documentation update
• 📚 Documentation update
• ♻️ Refactoring
• 🔧 Technical change

🔧 Changes

• ingestion: handle webhooks related to archive / delete events and call the web API
• web: add an API to archive offers by reference
• add an API key authentication for HTTP calls from ingestion to web
• add tests

🛸 Required dependencies for this change (if applicable)

🏝️ How to test (if applicable)

Steps to reproduce or test

📸 Screenshots (if applicable)

✅ Checklist

• 💅 I have added or updated the appropriate tests.
• 📝 I have updated or added the necessary documentation.
• 🚀 I have considered the impact on performance, security, and user experience.
• 👀 I have requested a review from a team member.

[Atheane]

Nice and quick implementation! Some suggestions about architecture:

Is it possible to add an application layer in ingestion, with a usecase archive_offers ? Also we could add DI to managers dependencies at this point.

For signature verification, maybe we could wrap this usecase with rbac guard, maybe in a custom decorator?

[AntoineAugusti]

@Atheane I have refactored the ingestion service to clean archi in a2e7379 and introduced a use case in 1b5d3f0

[vincentporte]

Very nice.
Ideally, for upcoming PR, we should decouple ingestion PR from tycho/web one, because we want to be able to deploy only the targeted app.

[AntoineAugusti]

@vincentporte Thank you for your review!

I've tackled your comments or asked questions. Let me know if you're ready to approve.

[vincentporte]

🚢 LGTM