Fixed issue with authentication using API token and Powershell version 6+#209
Open
sebmaurer wants to merge 214 commits into
Open
Fixed issue with authentication using API token and Powershell version 6+#209sebmaurer wants to merge 214 commits into
sebmaurer wants to merge 214 commits into
Conversation
…oString Overwrite ToString() method of custom classes
…OfCaptcha Warn in case the server requires a captcha
IMPROVEMENTS: - Shows a warning when the server requires a CAPTCHA for the authentication (AtlassianPS#91, [@lipkau][]) - Custom classes now print relevant data in `ToString()` (AtlassianPS#92, [@lipkau][])
…adAssembly Ensure System.Web is loaded
* PowerShell is not setting the encoding to UTF-8 for application/json * See PowerShell/PowerShell#5530
* Use Title as a parameter for the search instead of filtering the results * Add parameter to use CQL to search for Page
…pportUnicode Fix encoding of Invoke-WebRequest response
Add documentation of ConfluencePS classes
Query for page
* System.Web assmebly is not available on PowerShell Core
Support pwsh (PowerShell Core)
Sync the submodule for ConfluencePS on the homapage repo everytime the CI finishes successfully AtlassianPS/AtlassianPS.github.io#20
Exclude page body from Get-Page and Get-ChildPage calls
…crease_id_range Fix support for new cloud page ids by increase id range
Adding the Personal Access Token to Set-ConfluenceInfo
Converted this to a secure Token String for the web request
Author
|
Personally I would also recommend using SecureString throughout the whole module for the API token because in my workaround this transformation only happens in the middle of execution. |
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
10 participants
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.
Description
Just a quickfix for authentication with Confluence via API token when using Powershell version 6+.
It is required by Microsoft Web-Request to provide the header "Token" with a secure string of the corresponding token.
Microsoft Web-Request does not expect a header named "PersonalAccessToken" therefore removed this before calling it.
Motivation and Context
closes #208
Types of changes
Checklist