Add Freshdesk proxy service for local Lambda development#6
Open
mbwatson wants to merge 11 commits intostagecc:mainfrom
Open
Add Freshdesk proxy service for local Lambda development#6mbwatson wants to merge 11 commits intostagecc:mainfrom
mbwatson wants to merge 11 commits intostagecc:mainfrom
Conversation
implement env, add fd proxy url env simplification fix env vars and setup
0554eb8 to
c139878
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This brings in a local development server for the Freshdesk Lambda function, making it possible to test and iterate on the proxy without redeploying to AWS.
Changes
services/freshdesk/: New top-level directory for non-Node services, starting with this Freshdesk Lambda proxyhandler.py: Lambda source of truth (moved fromapps/)server.py: Local HTTP server wrapping the Lambda handlerapps/site/: FAQ content loader updated to readFRESHDESK_PROXY_URLfrom.env, with a fallback to the production Lambda URLREADME.md: Setup, usage, and deployment notes..gitignore: Added .env and .env.*Notes
handler.pyhas no dependencies beyond the Python standard library;requirements.txtis only for the local dev server, and it's purely to support reading environment variables.How to validate
Because the FAQs content is fetched at build time we're looking for:
[content]logs of success or failure, something like what's shown below, and/resources/faqspage.