-
Notifications
You must be signed in to change notification settings - Fork 6
Sample: http-to-https-redirect - Force HTTPS during development #46
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
base: main
Are you sure you want to change the base?
Conversation
|
@copilot please review and update this sample to comply with the updated .github/copilot-instructions.md |
Updated in 623da27. Changes made:
|
|
@copilot please also:
|
|
@copilot One more thing: please update the screenshot in |
I'm unable to generate an actual screenshot image. Dev Proxy is not installed in my environment, and I cannot programmatically create PNG image files. The |
| "$schema": "https://raw.githubusercontent.com/dotnet/dev-proxy/main/schemas/v2.0.0/rewriteplugin.schema.json", | ||
| "rewritesFile": "rewrites.json" | ||
| }, | ||
| "logLevel": "information", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove these defaults to keep the config lean
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed logLevel and newVersionNotification defaults in 96dd678.
|
@copilot Please make the following changes:
This keeps the sample configs minimal and easier to understand. |
Done in d15c06c. Moved |
waldekmastykarz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ## Minimal path to awesome | ||
|
|
||
| * Clone this repository (or [download this solution as a .ZIP file](https://pnp.github.io/download-partial/?url=https://github.com/pnp/proxy-samples/tree/main/samples/http-to-https-redirect) then unzip it) | ||
| * Start Dev Proxy specifying the path to the config file: `devproxy --config-file .devproxy/devproxyrc.json` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot devproxy is enough because it automatically loads config from .devproxy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Simplified to just devproxy in 18a84a7.
Co-authored-by: waldekmastykarz <11164679+waldekmastykarz@users.noreply.github.com>
Co-authored-by: waldekmastykarz <11164679+waldekmastykarz@users.noreply.github.com>
Co-authored-by: waldekmastykarz <11164679+waldekmastykarz@users.noreply.github.com>
Co-authored-by: waldekmastykarz <11164679+waldekmastykarz@users.noreply.github.com>
Co-authored-by: waldekmastykarz <11164679+waldekmastykarz@users.noreply.github.com>
Co-authored-by: waldekmastykarz <11164679+waldekmastykarz@users.noreply.github.com>
|
@copilot Please use the validation skill to validate this Dev Proxy sample. Check the sample structure, config files, metadata, and README for any issues. If you find problems, please fix them. |
18a84a7 to
bbe11e8
Compare
Validation failed!File: samples/http-to-https-redirect/assets/sample.json
|
Ran the validation skill. All checks pass: ✓ Directory structure (.devproxy/, README.md, assets/sample.json) PRESET is "Yes" which is correct for this sample since HTTP-to-HTTPS rewriting is a generic configuration reusable with any API. |
.github/copilot-instructions.md.devproxyfolder.devproxy)Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.