Skip to content

Feature/ch21518/filecoin#45

Open
jsonsivar wants to merge 4 commits intomasterfrom
feature/ch21518/filecoin
Open

Feature/ch21518/filecoin#45
jsonsivar wants to merge 4 commits intomasterfrom
feature/ch21518/filecoin

Conversation

@jsonsivar
Copy link
Copy Markdown
Contributor

No description provided.

@shortcut-integration
Copy link
Copy Markdown

This pull request has been linked to Clubhouse Story #21518: Space - Filecoin BE.


const result = await this.query(params);

console.log('getIdentitiesByDisplayName: ', result);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added these logs to troubleshoot another issue. Will clean up once that is fixed.

auth: {
type: 'AWS_IAM',
credentials: {
// is there a way to get these from the lambda role
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This piece I am not hundred percent on. Will these env vars get populated or maybe there is a way to set this auth to use the default lambda execution role? That's what it should be because the permissions in serverless.yml are set for that.

Subject: { Data: emailBody.subject },
},
Source: emailBody.from || 'Space <hi@space.storage>',
Source: emailBody.from || 'Space <hey@space.storage>',
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated to Filecoin and this fix is already in dev/prd.

@jsonsivar jsonsivar requested a review from gpuente February 26, 2021 13:57
Copy link
Copy Markdown

@nandit123 nandit123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants