Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add agent shell #246

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Add agent shell #246

wants to merge 29 commits into from

Conversation

LeonRomo
Copy link
Collaborator

This is a mere skeleton of the agent command

@LeonRomo LeonRomo added the Bug label Oct 16, 2023
@LeonRomo
Copy link
Collaborator Author

Authentication issue here. seems like its coming from OnTask. here are the possible issues that I formulated:

  1. Not sure if the endpoint does not require session authentication instead of token authentication while I am providing a bearer token, the server may expect a session-based login, especially if it's a local development server with Django's default settings.
  2. OnTask expects the token to be used with the Bearer schema. Although this is a common standard, some APIs may use a different method.
  3. The token has the right permissions to access the /table/3/pmerge/ endpoint. The token might be valid but might not have the scope required for the requested operation.
INFO:root:Authenticated with Canvas successfully!
Starting to fetch data for course 396, quiz 391
Finished fetching data.
ERROR:root:Failed to merge OnTask table: 401, {"detail":"Authentication credentials were not provided."}
ERROR:root:Failed to update the OnTask table with data from Canvas.
INFO:root:Request to http://127.0.0.1:8000/table/3/merge with headers {'accept': 'application/json', 'Authorization': 'Bearer 96ceb3b04c66be6b1ba60aca9e7ac09fcc153509'} returned 401
ERROR:root:Error fetching OnTask data: 401, Response: {"detail":"Authentication credentials were not provided."}
ERROR:root:Error fetching OnTask data: 401, {"detail":"Authentication credentials were not provided."}
INFO:root:Data has changed. Proceeding with OnTask update.
ERROR:root:Failed to merge OnTask table: 401, {"detail":"Authentication credentials were not provided."}
ERROR:root:OnTask table update failed.
Agent operation completed.

@abelardopardo abelardopardo added postpone Will be considered at a later time and removed Bug Implementation in progress labels Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
postpone Will be considered at a later time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants