Skip to content

#937, #921 improve evaluation of proxy configuration, allow empty port#951

Merged
helloanoop merged 3 commits intousebruno:mainfrom
mirkogolze:bugfix/correct-proxy-settings-evaluation
Nov 18, 2023
Merged

#937, #921 improve evaluation of proxy configuration, allow empty port#951
helloanoop merged 3 commits intousebruno:mainfrom
mirkogolze:bugfix/correct-proxy-settings-evaluation

Conversation

@mirkogolze
Copy link
Contributor

If no proxy configuration is given on collection level, check for the global proxy configuration.
Allow empty port for proxy configuration

Contribution Checklist:

  • The pull request only addresses one issue or adds one feature.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or gifs to help explain the change if applicable.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

const dataBuffer = Buffer.from(response.data);
// Parse the charset from content type: https://stackoverflow.com/a/33192813
const charset = /charset=([^()<>@,;:\"/[\]?.=\s]*)/i.exec(response.headers['Content-Type'] || '');
const charset = /charset=([^()<>@,;:"/[\]?.=\s]*)/i.exec(response.headers['Content-Type'] || '');
Copy link
Contributor

Choose a reason for hiding this comment

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

@mirkogolze Is the removal \ intentional ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Inside brackets [...] you don't need to escape "
image
image

And my IntelliJ really helps me here.

@helloanoop helloanoop merged commit 11ce057 into usebruno:main Nov 18, 2023
@helloanoop
Copy link
Contributor

Thanks for taking care of this @mirkogolze !

bijin-bruno added a commit that referenced this pull request Feb 13, 2026
* feat: button storybook

* feat: update button component with new rounded options and story

* fix: pasting request ito parent folder even if request is selected (#6446)

* Add right-click context menu to request tabs with MenuDropdown # (#6502)

* refactor: replace Dropdown with MenuDropdown in RequestTab component; update Dropdown props handling in Dropdown component

* refactor: remove Portal import and simplify menuDropdown rendering in RequestTab component

* refactor: streamline RequestTabMenu functionality and improve tab closing methods with async handling

* refactor: enhance Dropdown and MenuDropdown components with improved props handling and styling adjustments

* refactor: enhance Dropdown and MenuDropdown components by improving structure and removing unused styles

* refactor: update Dropdown and MenuDropdown components to append to sidebar sections container for improved layout

* refactor: integrate dropdownContainerRef for improved MenuDropdown positioning in RequestTabs and Sidebar components

* refactor: update Dropdown component to include 'tippy-box' class for e2e test selections

* refactor: update dropdown item selection logic in selectRequestPaneTab function for improved accuracy

* refactor: add fixed positioning to popperOptions in Collection and CollectionItem components for improved dropdown behavior

---------

Co-authored-by: sanjai <sanjai@usebruno.com>

* export & import in opencollection format (#6329)

---------

Co-authored-by: Anoop M D <anoop.md1421@gmail.com>
Co-authored-by: Anoop M D <anoop@usebruno.com>
Co-authored-by: Pooja <pooja@usebruno.com>
Co-authored-by: Abhishek S Lal <abhishek@usebruno.com>
Co-authored-by: sanjai <sanjai@usebruno.com>
Co-authored-by: naman-bruno <naman@usebruno.com>
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.

2 participants