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

feat: https detection #260

Merged
merged 5 commits into from
Jun 14, 2019
Merged

feat: https detection #260

merged 5 commits into from
Jun 14, 2019

Conversation

blowsie
Copy link
Contributor

@blowsie blowsie commented Jun 13, 2019

detect if nuxt server.https is set, assume that https is being used, preventing CORS errors and streamlining configuration

detect if nuxt `server.https` is set, assume that https is being used, preventing CORS errors and streamlining configuration
@atinux atinux requested a review from pi0 June 13, 2019 11:03
@atinux atinux self-requested a review June 13, 2019 11:05
@codecov
Copy link

codecov bot commented Jun 13, 2019

Codecov Report

Merging #260 into dev will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@         Coverage Diff         @@
##            dev   #260   +/-   ##
===================================
  Coverage   100%   100%           
===================================
  Files         1      1           
  Lines        27     28    +1     
  Branches     12     13    +1     
===================================
+ Hits         27     28    +1
Impacted Files Coverage Δ
lib/module.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c4e73d0...6cc235c. Read the comment docs.

Copy link
Member

@atinux atinux left a comment

Choose a reason for hiding this comment

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

Sounds good to me, should we add a test case for it?

@blowsie
Copy link
Contributor Author

blowsie commented Jun 13, 2019

Oops i made a mistake this.options.server && this.options.server.https would resolve to the config object, not a boolean it should have a double bang !! or similar depending on your coding style.

lib/module.js Outdated Show resolved Hide resolved
lib/module.js Outdated Show resolved Hide resolved
blowsie and others added 2 commits June 14, 2019 08:41
Co-Authored-By: Alexander Lichter <manniL@gmx.net>
Co-Authored-By: Alexander Lichter <manniL@gmx.net>
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.

4 participants