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

Prevent "Blocked host:" message #336

Merged
merged 4 commits into from Oct 2, 2021
Merged

Conversation

PromoFaux
Copy link
Contributor

@PromoFaux PromoFaux commented Oct 2, 2021

When using the latest container, I came up against the below issue, which was preventing Monzo webhooks from being processed.

image

Initially I just mounted the development.rb file and hardcoded config.hosts << myhostname.com, but that seemed really lazy, so figured I'd find a better way to do it that others may benefit from.

I also have made some tweaks to the GHA config so that it can be pushed to ones own repository

Proof of the GHA config working: https://github.com/PromoFaux/fintech-to-ynab/runs/3776908590?check_suite_focus=true

(I've also no idea if .nil? is the correct way to do this, I stackoverflowed my way to it, not knowing Ruby)

 - Update to reflect master branch name change to main
 - GITHUB_TOKEN is now valid for pushing to ghcr.io
 - Allow forked repos to utilise action script and push to their own package repository - requires lowercaseing the github.repository context

Signed-off-by: Adam Warner <me@adamwarner.co.uk>
…st:" message when attempting to connect from something other than localhost / 0.0.0.0

Signed-off-by: Adam Warner <me@adamwarner.co.uk>
Signed-off-by: Adam Warner <me@adamwarner.co.uk>
Signed-off-by: Adam Warner <me@adamwarner.co.uk>
@scottrobertson scottrobertson merged commit 713ba4b into syncforynab:main Oct 2, 2021
@scottrobertson
Copy link
Collaborator

@PromoFaux i merged this, but it broke the Docker stuff. It needs the GHC_TOKEN stuff, as the default token does not have enough permissions to push to the registry.

Can you revert those parts in another PR?

@PromoFaux
Copy link
Contributor Author

Hmmm, that's odd, I made sure it worked before I PR'd - the image pushed to promofaux/fintech-to-ynab

I wonder what stops it working here? Will have a play about tomorrow!

@PromoFaux
Copy link
Contributor Author

Ah. I wonder...

What is the Workflow permissions set to in https://github.com/syncforynab/fintech-to-ynab/settings/actions?

On my fork it has defaulted to Read and write permissions

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.

None yet

2 participants