This PR moves the request type and method constants to the ServiceRequest object instead of the LtiServiceConnector. I think it makes more sense for them to live here, since they're describing the request itself, not anything about the connector.
This also fixes a bug where we were treating every request like an auth request in the logs.
Testing
[x] I have added automated tests for my changes
[x] Confirmed that this passes E2E tests on the platform
Summary of Changes
This PR moves the request type and method constants to the
ServiceRequestobject instead of theLtiServiceConnector. I think it makes more sense for them to live here, since they're describing the request itself, not anything about the connector.This also fixes a bug where we were treating every request like an auth request in the logs.
Testing
Here's the branch: https://github.com/packbackbooks/questions/pull/6533