Improve debugging/developer experience with rest_log module

You cannot add dev-dependencies in libraries in composer. well you can, but it would have no affect.
I think it might work to "suggest" it via composer suggest option + we can add it to our documentation.

I added the rest log in require-dev and suggest in composer.json.

For checking api requests I've added a request check:

  protected function isApiResponse(Request $request): bool {
    return $request->attributes->get('lupus_ce_renderer') || $request->getRequestFormat() == 'custom_elements';
  }

I've also been considering some other ways, but this seemed the most straightforward. The module now works just on on/off - if enabled it logs and if not it doesn't. I think that for development experience this is enough.

@fago @useernamee can you guys familiarize with my last comment in rest log mirror issue( https://www.drupal.org/project/rest_log/issues/3432400#comment-15563623 ✨ Add support for Lupus Decoupled Drupal Fixed ) and update your merge request with needed changes?

Implemented the change from ✨ Add support for Lupus Decoupled Drupal Fixed

looks all good to me! however, the gitlab pipeline is failing with the changes. we need to make it accept it.

I wasn't aware that ✨ Add support for Lupus Decoupled Drupal Fixed was not released yet. I think we should be good now - at least for the CI part. I kind of didn't want to put the patch in the lupus_decoupled composer.json file because than I'd also have to add a patch as well.

it's merged now, so we should do the same and remove the patch. Could someone update + verify it's all good, so we can move and merge this?

I removed that rest_log patch and checked the gitlab workflow.

Anyways I had to put the patch back in place. The issue is that rest_log last release was in 2023. ✨ Add support for Lupus Decoupled Drupal Fixed was merged in April 2024.

I've contacted maintainers of REST Log module and asked them if they can create a new release. That's the easiest way forward, since if we add a patch we will need to remove it after new release.

@useernamee published new release.

thank you!

tested locally. requests are being logged. I'm waiting for green pipeline.

all green, merged.

Automatically closed - issue fixed for 2 weeks with no activity.