Login paths not working for all except Microsoftaccount

I have set up a test web app with microsoftaccount, facebook and twitter access. The login pages at /.auth/login/facebook and /.auth/login/twitter are returning 404s.  The microsoft account endpoint works.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d4518495-4e07-ce2e-4045-5a50b7922b55
Version Independent ID: 034d1ba0-a737-21ab-3717-ed8798ac5fe1
Content: Advanced usage of authentication and authorization - Azure App Service
Content Source: articles/app-service/app-service-authentication-how-to.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Hi @AndyEdmonds, thanks for the feedback! We are currently investigating and will update you shortly.

Hi Grace,
This turns out to be caused by a misunderstanding of how your blade in the Azure portal works. I saved the new credentials for each added service. But I didn’t see that I had to then click the save button on the whole blade to save the changes. It now works.
Andy
From: Grace-MacJones-MSFT<EMAIL_ADDRESS>Sent: 30 March 2019 03:53
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Andrew Edmonds<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Login paths not working for all except Microsoftaccount (#28363)
Hi @AndyEdmondshttps://github.com/AndyEdmonds, thanks for the feedback! We are currently investigating and will update you shortly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/28363#issuecomment-478203671, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMB1zBk0rzSo1wbCzR8nGasEduqE4Wjtks5vbt-OgaJpZM4cTD61.

Hi Grace,
On further investigation as to why I thought I’d set up facebook and twitter accounts in the Azure authentication blade and hadn’t, the following process occurs:
If you set up facebook, and supply valid credentials, but don’t set any scopes, the blade permits you to OK the entry, without warnings, and the facebook row  reports “configured” but the save button for the whole settings blade is disabled.
This leads you to believe you have created a valid facebook connection. In reality you haven’t until you set scopes.
Andy
From: Grace-MacJones-MSFT<EMAIL_ADDRESS>Sent: 30 March 2019 03:53
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Andrew Edmonds<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Login paths not working for all except Microsoftaccount (#28363)
Hi @AndyEdmondshttps://github.com/AndyEdmonds, thanks for the feedback! We are currently investigating and will update you shortly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/28363#issuecomment-478203671, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMB1zBk0rzSo1wbCzR8nGasEduqE4Wjtks5vbt-OgaJpZM4cTD61.

@AndyEdmonds  thanks for the update! we are glad you were able to resolve your issue.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Explore: Manage Invites: Claimed

Is your feature request related to a problem? Please describe.
User does not have a way to see which users have accepted their invitations.
Describe the solution you'd like
Once the user has selected 'Manage Invites, they will see two categories they can select between:

'Claimed Invites' (users that have accepted the invitation and are onboarded). and
'Unclaimed Invites' (invitations that have not been claimed and are still open).

If the user has no claimed invitations, the user will see text that states 'No Invitations have been claimed'.
On the bottom they will see a button to Create Invite. This will take the user to the 'Invite a Friend' Module.

When the user has invitations that have already been claimed, the user will see a list of the users that have claimed the invitation along with their Citizen status (Visitor, Resident, Citizen)

❌ Test Case 1 (see claimed invite counts)

Login with an account that has claimed invites.
Tap Explore > Invite a Friend > (gear icon).

Current behavior
Observe no invite count: Claimed Invites.
Expected behavior
Observe claimed invite count, e.g. Claimed Invites (10).
✅ Test Case 2 (see claimed invites)

Login with an account that has claimed invites.
Tap Explore > Invite a Friend > (gear icon).
Observe claimed invites along with name, username and citizen status.

✅ Test Case 3 (see 0 claimed invites)

Login with an account that has no claimed invites.
Tap Explore > Invite a Friend > (gear icon).
Observe No Invitations have been claimed.
Tap Create Invite button.
Observe a new invite.

@gguijarro-c-chwy Test Case 1 is failing.

@JGDoh Nitpicks:

Invitations in No Invitations have been claimed is capitalized. Intentional?
As a possibility, perhaps we want to say You have no claimed invitations, to sync up with Unclaimed Invites > You have no unclaimed invitations?

✅ Test Case 1 (see claimed invite counts)

Login with an account that has claimed invites.
Tap Explore > Invite a Friend > (gear icon).

Current behavior
Observe no invite count: Claimed Invites.
Expected behavior
Observe claimed invite count, e.g. Claimed Invites (10).
✅ Test Case 2 (see claimed invites)

Login with an account that has claimed invites.
Tap Explore > Invite a Friend > (gear icon).
Observe claimed invites along with name, username and citizen status.

@JGDoh Do we want to make a separate ticket for this failing test case? Or let it go? 🤓
❌ Test Case 3 (see 0 claimed invites and create a new invite)

Login with an account that has no claimed invites.
Tap Explore > Invite a Friend > (gear icon).
Observe No Invitations have been claimed.

Current behavior
No Create Invite button is present.
Expected behavior
4. Tap Create Invite button.
5. Observe a new invite.

Let's let it go for now. I'll create a ticket for this task to be released for 2.5.0.
As for the No invitations have been claimed that's a typo. We can keep it lowercase.
"You have no unclaimed invitations" is definitely better here. Let's change to fix that. If not this release than the next is fine.
@buildc0de


TaskController

TaskController should allow the user to clear an error message. Currently this is done only when a new task is created or a null is assigned as the current task.

Old issue. Seems to have been addressed already


implement new AddPermissionWithContext func within TestSNSAPI

This will add unimplemented functions for the TestSNSAPI interface to make tests pass. If this build passes should I also add the other context funcs?

Coverage increased (+0.3%) to 40.909% when pulling 16a7e6b1da15f917a342cd6b15f588d30eb6ad36 on scotts-nyt:master into 12c3a2a0a5dcc7cdc6f3d13b1ceae135872f786a on NYTimes:master.

Coverage remained the same at 40.59% when pulling 399058870f13efe2f6469e578349dbc722a03239 on scotts-nyt:master into 12c3a2a0a5dcc7cdc6f3d13b1ceae135872f786a on NYTimes:master.

Coverage remained the same at 40.59% when pulling f341875831ffaf4c119fac00c1b672d942e90519 on scotts-nyt:master into 12c3a2a0a5dcc7cdc6f3d13b1ceae135872f786a on NYTimes:master.

Coverage remained the same at 40.59% when pulling 1fac6f54e1317df512bcb469accb0a079d22fb53 on scotts-nyt:master into 12c3a2a0a5dcc7cdc6f3d13b1ceae135872f786a on NYTimes:master.

Coverage remained the same at 40.59% when pulling d5e946323c18489afb771e44130156287c256aff on scotts-nyt:master into 12c3a2a0a5dcc7cdc6f3d13b1ceae135872f786a on NYTimes:master.

Coverage remained the same at 40.59% when pulling 876119a80c106dae7def691426968775d0b9f781 on scotts-nyt:master into 12c3a2a0a5dcc7cdc6f3d13b1ceae135872f786a on NYTimes:master.

Coverage remained the same at 40.59% when pulling 9e8d78a460795975844ad45aaddb9e11d7f39644 on scotts-nyt:master into 12c3a2a0a5dcc7cdc6f3d13b1ceae135872f786a on NYTimes:master.

We pulled this in with PR #95


Poll on events for file reading

When multiple connections are monitoring events via the remote API, the inotify in the hpcloud library seems unable to consistently send events.  Switching from inotify to poll seems to clear this up.
Fixes: #6664
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

LGTM. Simple fixes are always nice

/lgtm
/hold

/hold cancel


[TCP-11] Add test setup file, resolving 'Invalid chai property' error

This fixes the 'Invalid Chai Property' error.
Related JIRA Ticket https://techtankto.atlassian.net/browse/TCP-11

Circular dependencies have been found within the sanity folder which is causing next build to fail. This will be addressed within the following ticket.


Reference install solution in issue #718 in install docs

Closes https://github.com/airbnb/react-native-maps/issues/718


一直报错Connection closed while receiving data

Connection closed while receiving data
pub get failed (server unavailable) -- attempting retry 1 in 1 second...

网络问题


Fix incorrect behaviour of number validator in some locales

Q
A

Is bugfix?
yes/no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#10488

Completion of the work started in PR #10488 with the modifications requested by @samdark

After many tests, I'm not convinced that this is a reliable solution.
I'm closing the PR.


Permission denied while it Allowed! access to external storage problem

I tried the sample app on Android 10 (AVD virtual device), and it failed to saving photo on device, "permission denied", while I Allowed it.
I search and found that this deprecated function is the problem.
https://github.com/burhanrashid52/PhotoEditor/blob/e94dd4b595f8d3d965824a1492e45a04fee744d7/app/src/main/java/com/burhanrashid52/photoeditor/EditImageActivity.java#L266
I replace it with:
File file = new File(this.getExternalFilesDir(DIRECTORY_PICTURES)                     + File.separator + ""                     + System.currentTimeMillis() + ".png");
and it works fine, but it save photo in app private folder.

I read that android 10 will not give direct access to external storage.

there are some workaround solutions, but I didn't use it.

Yes. I am aware of it. If would be really great if you can help to raise a PR for Android 10 storage support

FWIW, I was able to save it to the DCIM folder in a custom app I'm writing.
    String FOLDER_NAME = "Subfolder";
    File directory = new File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DCIM), FOLDER_NAME);
    if (!directory.exists()) {
        exists = directory.mkdirs();
    }

    File file = null;
    if (exists) {
        file = new File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DCIM)
                + File.separator + ""
                + FOLDER_NAME
                + File.separator + ""
                + System.currentTimeMillis() + ".png");
      }

I also added a media scan in the onSuccess method so the new file shows up immediately in the gallery apps
    MediaScannerConnection.scanFile(EditImageActivity.this, new String[] { finalFile.getAbsolutePath() }, null, null);

I then pass on the imageUri back to the calling activity where in onActivityResult I ask for permissions so the app can read/write the newly written file:
    final int takeFlags = data.getFlags()
        & (Intent.FLAG_GRANT_READ_URI_PERMISSION
        | Intent.FLAG_GRANT_WRITE_URI_PERMISSION);

    getContentResolver().takePersistableUriPermission(uri, takeFlags);

Hope that might help a bit.

thanks, @swentel, @burhanrashid52

I just Opened PR about this Issue,
My fix:
app:manifest.xml
<application
        android:name="com.burhanrashid52.photoeditor.PhotoApp"
        ........
        android:requestLegacyExternalStorage="true">

EditImageActivity
Replace
 File file = new File(Environment.getExternalStorageDirectory()
                    + File.separator + ""
                    + System.currentTimeMillis() + ".png");

with
final File file = new File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_PICTURES),
                System.currentTimeMillis() + ".png");

then add this method
//added the photo file to gallery
    private static void galleryAddPic(Context context, String imagePath) {
        Intent mediaScanIntent = new Intent(Intent.ACTION_MEDIA_SCANNER_SCAN_FILE);
        File f = new File(imagePath);
        Uri contentUri = Uri.fromFile(f);
        mediaScanIntent.setData(contentUri);
        context.sendBroadcast(mediaScanIntent);
    }

call the method after in saved onSuccess
    @Override
     public void onSuccess(@NonNull String imagePath) {
        ...........
        mPhotoEditorView.getSource().setImageURI(mSaveImageUri);
        galleryAddPic(EditImageActivity.this, file.getAbsolutePath());  <-- here
       }

I hope this helps someone

@breel93 Thanks bro. Helped a lot :D

Hi.. i'm still experiencing permission denied... while permission is already given in manifest... and also tried solution above but.. still permission is denied

check whet

Hi.. i'm still experiencing permission denied... while permission is already given in manifest... and also tried solution above but.. still permission is denied

check whether u give permission to access storage from mobile setting for this app..

Fixed in #326


Script crash after first Attack button press

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
NA
FGA build number
1534
Describe the bug
Script crash after first Attack button press and scan of the command cards.
null

c4.a$b
	at c4.a.a(AutoBattle.kt:3)
	at a5.c.q(EntryPoint.kt:3)
	at o5.a.run(Thread.kt:1)

Video
https://user-images.githubusercontent.com/4147075/138022185-83bdeb34-3899-4d2d-8be8-86320d2371ab.mp4
Device model
BlueStacks Hyper-V/Samsung Galaxy S8 Plus
Android version
7
Screen size
1920x1080
RAM
8GB

The crash is due to not being able to match servant face card I believe due to buff icons, similar to #913.
But, now the code was assuming something would match, and when nothing matched, it crashed.
ToDo for me:

Add TeamSlot.Ghost for cases when servant cannot be matched.
Adjust the face card crop region to exclude buff icons to prevent matching failure.

Just got this too, coincidentally after updating to the latest canary.
null

c4.a$b
	at c4.a.a(AutoBattle.kt:3)
	at a5.c.q(EntryPoint.kt:3)
	at o5.a.run(Thread.kt:1)

Is it just due to Space Ishtar?

lastest build

1534 was latest until yesterday. Try 1536.
Accessibility issue is unrelated, there's a pinned issue. That's something you'll have to follow the steps to solve for yourself.


Not able to handle public fields without getter and setter

If there is a field declared as public, but without getter and setter methods, Selma is not able to generate the mapping code for it.

For now Selma only works with standard Java bean. It wouldn't need huge work for Selma to support this kind of mapping.


Add an annotation storage in order to save annotation data in acroforms

The goal of this storage is to be able to save form data from the html layer and then be able to print and save them.

/botio test

First part done; thank you!


Mouse and ui buttons misaligned

When I run the mav demo at first glance the buttons appear not to work. On closer inspection they do but the mouse must be about half the screen down from where the button actually is and then the button will be highlighted
I have placed an arrow in the screenshot approximately where the mouse was.

With a bit of patience it is possible to use the ui but obviously the mouse is not where the program thinks it is.
I am running jdk 11  64bit
windows 10 with a
resolution of 4k. on primary screen 2k on second screen.
If I drag the window from the primary monitor to the secondary monitor then the buttons highlight as they should when the mouse is over them.
Interestingly if you drag the screen so that it straddles the two monitors with >50% of the display on the secondary monitor but the buttons on the primary monitor (primary screen to left of secondary) everything lines up fine.
As soon as there is more than 50% of the display on the primary monitor the cursor is positioned incorrectly again.
A ui lib bug perhaps?
To run I opened intellij (2020.3) cloned from the git repo and just hit run.
Apart from the issue with the mouse being in the 'wrong' place the program seems to work as intended and does not generate any exceptions or other errors. (A few warnings on startup but thats it, an example below)
`"WARNING:` The texture /Models/classic_motorcycle/textures/MTA_Metales_metallicRoughness.png has linear color space, but the material parameter LightMap specifies no color space requirement, this may lead to unexpected behavior.
Check if the image was not set to another material parameter with a linear color space, or that you did not set the ColorSpace to Linear using texture.getImage.setColorSpace()."

A ui lib bug perhaps?

That's a reasonable guess.
Would you do one thing for me, please? At the command line, cd to your repo and then do a git rev-parse HEAD and report back with the output. That way I can be sure which libraries were in use.

Hi Stephen, The result of that is
07549eecf38f25c9cd811e59d93a84e99083081c

So your build was using JMonkeyEngine v3.5.0-beta5. Good to know!
If you don't want to test beta software (you just want to run the demo) I suggest you follow the instruction in the README:
https://github.com/stephengold/jme-vehicles/blob/master/readme.md#build
which will check out tag "project-1.5.0" (hash a84846b), which uses JMonkeyEngine v3.4.0
Please let me know if "project-1.5.0" has the misalignment issue.

project-1.5.0 is absolutely fine. No issue with misalignment there

Thanks for letting me know!

@jamessbull I can't reproduce this issue on my system because I only have one monitor. Will you have time to test some possible fixes during the coming week?

Probably related to https://github.com/jMonkeyEngine/jmonkeyengine/issues/1745

In which case, 1c86e45e should solve this issue.


Prometheus metrics counts a client cancel as a 500

Do you want to request a feature or report a bug?
Bug report

What did you do?
Start traefik and proxy to a service ansering slowly.
Example slow service:
package main
import (
	"net/http"
	"time"
	"github.com/gin-gonic/gin"
	"github.com/koding/multiconfig"
)
type config struct {
	Port  string `default:"8080"`
	Sleep int    `default:"11"`
}
func main() {
	config := config{}
	multiconfig.New().MustLoad(&config)
	r := gin.Default()
	r.GET("/goslow", response(config))
	r.POST("/goslow", response(config))
	r.PUT("/goslow", response(config))
	r.DELETE("/goslow", response(config))
	r.Run(":" + config.Port)
}
func response(c config) gin.HandlerFunc {
	return func(g *gin.Context) {
		time.Sleep(time.Second * time.Duration(c.Sleep))
		g.Status(http.StatusOK)
	}
}

Do request from a browser and before the service answers (before 11 sec) press escape or Stop in the browser
What did you expect to see?
Traefik should not report this as 500 errors.
What did you see instead?
Traefik will increae the 500 code counters:
traefik_requests_total{code="500",method="GET",service="/goslow"} 1

Output of traefik version: (What version of Traefik are you using?)

Any configuration should work just fine.

Duplicate of #1926


fix nx registration e2e local

@marcellmueller , I grabbed the commits from this PR to help me with this PR: https://github.com/bcgov/cas-registration/pull/1604. So this PR may not be needed anymore


Use uaal to improve tunny ui

Is your feature request related to a problem? Please describe.
Use uaal to improve tunny ui
https://docs.unity3d.com/ja/current/Manual/UnityasaLibrary-Windows.html
Describe the solution you'd like
Describe alternatives you've considered
Additional context

maybe wpf is more fit


[PR] Dark Mode

Description
Dark Mode 🦇 🌔 🎉 This is an MVP of what should eventually be a more holistic, app-wide change (right now it just changes the sidebar colors). I've added a toggle in the settings under Appearance, and modified the colors in the sidebar appropriately. I do think that this should be merged in the meantime, since building out the "dark theme" will be much easier once the logic has been added.
Note: I'm not sure how the i18n stuff works within this project. In the settings it renders as !!!Dark Mode, which I understand to mean that there is no available translation?

Motivation and Context
Fixes #49. It's been the only thing that hasn't totally sold me on Franz since this afternoon when I was first introduced to it, so I'm happy it wasn't too difficult to implement 😄
How Has This Been Tested?
There aren't any tests in the repo, so I've just meandered around on my own within the app.
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project (run $ yarn lint).
[x] I have updated the documentation accordingly.

☝ Not sure if this should be documented?

@haraldox any news on this?

Hi is there anything happening with this? I would really like this feature.

waiting to happen

This would be awesome!

@JasonEtco it looks like this is going to be accepted based on the approvals above... want to update your branch to fix up the conflicts? :)

@whitfin those approvals aren't from maintainers of the project so while its nice to know people approve of it, its not really meaningful 🤷♂
Quite a lot has changed since I opened this PR and I don't feel comfortable trying to wade through the conflicts (nor do I have the bandwidth for it). Maintainers, y'all are welcome to take over this branch and make whatever changes are needed, or close it out if it isn't a priority.

@whitfin actually, it looks like this PR will never be merged. Which is sad. It seems like Franz is not under active development any more : last commit from @adlk dates back to more than two weeks ago, whereas last commit from @haraldox is almost one and a half month ago.

@alecive what you mean with not under development anymore? I saw that there's a "pro" mode, i don't think the development can be stopped whenever they want to, especially if they have some paying users (i'm not one of them, i'm just curious about your statement) :)

I saw a comment in Slack that @adlk is currently OOO and will be back when.. he's back. No need to worry - yet.

I also add styling to the title bar since I'm on Windows, and when I open up Franz, I hit, ctrl+alt+i and run this quick script:
var cssStyle = document.createElement('style'); cssStyle.type = 'text/css'; var minCss = '#ReactVirtualized__Grid,#electron-app-title-bar,.list-item,.menu-pane,.sidebar,.toolbar-dropdown.open .button-component{background:#222!important;box-shadow:1px 0 5px rgba(0,0,0,.38)!important}#electron-app-title-bar,.list-item,.toolbar-dropdown.open .button-component{color:#d3d3d3!important}.tab-item.is-active{border-left:4px solid rgba(255,255,255,.4);background:rgba(255,255,255,.15)}',  rules = document.createTextNode(minCss); cssStyle.appendChild(rules); document.getElementsByTagName("head")[0].appendChild(cssStyle);
For anyone interested in the beautified CSS (yes, I used !important):
#electron-app-title-bar,  .toolbar-dropdown.open .button-component,  .menu-pane,  .list-item,  #electron-app-title-bar,  #ReactVirtualized__Grid,  .sidebar{     background:#222 !important;     box-shadow:1px 0 5px rgba(0,0,0,.38) !important; } .toolbar-dropdown.open .button-component,   #electron-app-title-bar,   .list-item{     color: lightgray !important; } .tab-item.is-active{   border-left:4px solid rgba(255,255,255,.4);   background:rgba(255,255,255,.15); }

@Cahl-Dee your solution is pretty neat - thanks for sharing.

Howdy folks, just checking to see if this is going to get merged.

any luck with this?

Looking forward to this!

Edit: I've added styling for the settings menu as well.
When I open up Franz, I hit, ctrl+alt+i and run this quick script (I also add styling to the title bar since I'm on Windows):
var cssStyle = document.createElement('style'); cssStyle.type = 'text/css'; var minCss = 'body{color:#d0d0d0}.badge{color:#000}#electron-app-title-bar,.list-item,.settings-navigation .settings-navigation__link,.toolbar-dropdown.open .button-component{color:#d3d3d3!important}#ReactVirtualized__Grid,#electron-app-title-bar,.list-item,.menu-pane,.settings .account .account__box,.settings-navigation,.settings__close,.settings__controls,.settings__header,.sidebar,.toolbar-dropdown.open .button-component{background:#222!important;box-shadow:1px 0 5px rgba(0,0,0,.38)!important}.settings__main{background:#333!important}.tab-item.is-active{border-left:4px solid rgba(255,255,255,.4);background:rgba(255,255,255,.15)}.service-table .service-table__row:hover,.settings .settings__close:hover,.settings-navigation .settings-navigation__link:hover{background:#000!important}', rules = document.createTextNode(minCss); cssStyle.appendChild(rules); document.getElementsByTagName("head")[0].appendChild(cssStyle);

Great, working perfect in the menus @Cahl-Dee
But is possible change the Whatsapp background and colors?

@StanSvec Same Here


Current working directory is changed

When saving the file to trigger the web browser refresh the vim current directory is changed to 'browserlink' plugin directory. This causes confusion when using :e or other plugins like ctrl-p that after each file save needs absolute paths or the project path to be restored.

oh! I didn't update, your right it is fixed thank you!


Disable logging for Ember client internal retry

This is coming from a user issue reported in Discord: https://discord.com/channels/632277896739946517/632286375311573032/902589984391983134

gtc_10001
—
Today at 11:10 AM
Hi, I'm using the Ember client (0.23.6), and I noticed that when the client automatically retries using an instance of Retry one of our custom sensitive headers that is normally redacted during logging by the middleware (via use of redactHeadersWhen) is being shown.  It seems that EmberClientBuilder allows you to specify a retry policy, but there's no way to specify a redactHeaderWhen function for the underlying instance of Retry. I ended up turning off that logger in my logging config but I ideally would like the retry logged without the sensitive header. Does anyone know of a way to do that with the current code?

The recommendation was that we disable logging for the internal retry since users are not even supposed to be aware of it.
Some alternatives:

Manually handcode the internal retry
Factor out a more reusable Retry interface which can invoke callback effects for the different states

No, I think we still want this.


Update global search to show marks bolder

Resolves "the searched word is not bold (good first issue)" of #2738
Global search result title are no longer marked.
mark tags are now bolder.

The marks should not be visible in titles as far as I undestood.

After reconsidering the desired Thanks for pointing that out
I'll update the change request accordingly

Add code to prevent marking in account case.


Request for the latest code

Hi,Ali
Recently, I have been studying the optimization of large language models to solve complex problems, and I hope to get the latest code for the article: OptiMUS-0.3: Using large language models to model and solve optimization problems at scale.
thanks

Hi @Vandomheim
We are in the process of cleaning up the code and publishing it.

It's now available on a separate branch. We will also add the documentation.

thanks a lot

Hi,
Is there any structure in the params.json and description.txt required for the problem, is OK to use natural language?
It would be better if there are a few examples of the question, which I find to be different from the v0.2.


Failed to fetch device status - same or similar to #1197

This form is for reporting bugs.  If you have encountered a bug, you should
generally have some log messages or device diagnostics to show something is
going wrong.  If your problem is more of a question or a request for help,
then Discussions is
probably a more suitable venue.
Describe the bug
Device goes "Off Line" in HA but is still working via the Tuya Smart app.
To Reproduce
Random, there is no fixed interval where this happens.
Only way to get them back online is to switch the power off and then on again.
Expected behaviour
A clear and concise description of what you expected to happen.
Additional context
The lights are always on.
This is only happening with my lights.
They are all the same type - A60 1800-2700K RGBWW light Brand is BNETA
If the bug involves a device, then please include device diagnostics from
Settings / Devices & Services / Tuya Local (your device) / "1 device"
See attached File - tuya_local-LS1.json.txt
tuya_local-LS1.json.txt
Please check if there are messages from Tuya Local in the Home
This error originated from a custom integration.
Logger: custom_components.tuya_local.device
Source: custom_components/tuya_local/device.py:548
Integration: Tuya Local (documentation, issues)
First occurred: 08:42:35 (25 occurrences)
Last logged: 08:44:08
Failed to fetch device status for LS1
If applicable, add screenshots to help explain your problem.

After i Cycled the power

Wireless networks are inherently unreliable, and Tuya devices may contain unknown bugs themselves.  If you are going to assert that there is a bug in the integration, then you need to come with evidence that can be acted on, not just speculation.  From the evidence you have presented, the bug is in the device that needs to be power cycled to recover.

So i did some more digging...
Ran TinyTuya on a seperate RPI and when the bulb goes offline in tuya-local(HA), TinyTuya is also not able to connect to the bulb.
I used "Fing" to try and ping the bulb, no response.
Checked for open ports, no open ports reported.
But the bulb is still working via the Tuya app.
So it seems that the Tuya bulbs "crash" and that you can't ping the device thus port 6668 is no longer accessible and that is why the bulb goes offline in HA.
Not sure what port is used by the Tuya app, but even though i am unable to ping the device, it still function via the Tuya app.

The Tuya app is able to access via the cloud when the local connection is not working.  The cloud is an outgoing connection, so even when the device is not accepting incoming requests, or responding to pings, it may be still connecting out to the cloud.  Perhaps it has changed its IP address.

When the bulb is in this state, are you able to do a packet capture on it to see if it's still initiating outbound connections to the Tuya cloud?

So i did some more digging... Ran TinyTuya on a seperate RPI and when the bulb goes offline in tuya-local(HA), TinyTuya is also not able to connect to the bulb. I used "Fing" to try and ping the bulb, no response. Checked for open ports, no open ports reported.
But the bulb is still working via the Tuya app. So it seems that the Tuya bulbs "crash" and that you can't ping the device thus port 6668 is no longer accessible and that is why the bulb goes offline in HA.
Not sure what port is used by the Tuya app, but even though i am unable to ping the device, it still function via the Tuya app.

Zodiak-69, I am experiencing the same or similar thing.  I have a couple devices in HA that seem to lose the TL connection but I can still control them through the app.  When I cycle the power all devices come back to the TL integration but randomly lose 1-3 lights devices.
Did you find a solution?

HiAnj-0110
I am still verifying, but initial results are promising.
I use Ubiquity AP's and looking on the forums, it seems that Tuya bulbs do not play well with WiFi.
On my setup, i have 3x Ubiquity AP's all with the same SSID.
This is what i have done so far and my Tuya bulbs are staying online so far.

All devices have Static IP's
Create a separate SSID for my IoT units, 2.4GHz only.
Disable the following:
a. Band Steering
b. BSS Transition
c. Roaming / Fast Roaming
d.PMF
e. Uplink connectivity Monitoring
f. Auto optimize network

So far it has made a huge difference.
I still want to do the following that was recommended as well.
X. Channel bandwidth to VTH20
Y. Set fixed channels to each AP, CH.1 and CH.11 is recommended.
Hope this can assist you

Hello, I also have this problem, my tuya sockets were randomly gone unavailable, I tried a lot, but the only thing that really helped me was changing the encryption on the WiFi to WPA only, now after several days I have only one socket which still throws "Failed to fetch device status" errors sometimes, I think it might be misunderstanding in protocol version, how do I find the right version to use with my socket? (BTW, they are two different sockets) Thanks in advance!

Just some feedback.
Since implementing the changes listed above - , - i have not had any of my Tuya bulbs go off line.
For now i am not making any changes until the end of March.

I have recently acquired a Ubiquite UDR and 2 AC lite AP's.  I have gone through and matched all the devices with static IP's on a dedicated 2.4 SSID and took your suggestions on the settings.  I will give it a couple days to see how things react and communicate.  I have a lot of TUYA, they were relatively cheap and were purchased prior to diving into HASS.
Thanks for the suggestions!

Just some feedback (2)
All my Tuya bulbs have now remained online and available in Tuya-local since i implemented the changed listed in my post above. Not once, did i have to power cycle my Tuya bulbs since i made the changes.
I think that i can say that the issues i had were all related to Tuya bulbs and their wifi implementation.
For me it was only my Tuya bulbs, they are all the same type (RGB Color Control + White Temp Control (Warm White 2700K ~ Cool White 6500K) 230V, 9W, 806lm) that was going off line, my other Tuya devices did not and were always available in HA.
So i think i can safely say that changing the settings has resolved my issues.
Next step is to move my devices to a VLAN.

So...
It has only been less than 24 hours and I am at a loss with the connectivity!?  I cannot figure out why I am losing connectivity, it is not just tuya lights.  I am not sure if it is the tuya-local integration, my wifi or other issues that I am not aware of.  When I log into the unifi-os web app, and go to the clients I have a range of poor to excellent status for the 60 devices connected through the UDR.  One specific device, a smart plug is 2' from the AP and the status is excellent but in the TL integration is bounces from connected to unavailable, why would this be? Also, the tuya lights, specifically INFIBRITE 6" RGBW, similar to your lights are unavailable randomly through HA but are connected in the UNIFI-OS.  Also to mention, I have connected all the devices through the SMARTLIFE app on my phone, when I check the connection it also shows them as offline.
You mentioned changing the bandwidth, I don't see where to do this in the unifi-os?
The below screenshot are just some of the lights that lose connection.

This screenshot is a light device that is next to the UDR and the status of its connection through the unifi-os

I don't know what else to try?  What am I doing wrong?  Any advice?

Hi Anj-0110
Not sure why your Tuya bulbs are going offline.
To set the bandwidth (This is a "Global" setting on your AP and will affect ALL SSID's),
Left hand menu: - Double round circle - Unifi Decices
Select the AP and click on the "Gear" icon.
2.4GHz, you can select: 20 or 40MHz as the bandwidth.
Next to that you can also select the WiFi CH you want to set the AP to use.
I do not run my UniFi software the whole time.
I only run it when i have issues, or want to make changes, else the UniFi network app is not running.

Hi Z-
Just sending an update...
I have taken a little different approach and have had success so far.  One critical sequence I didn't perform prior to my previous post was power cycle the devices.  With your setting suggestions HA was seeing all the devices and seemed to hold the connection.  However, after some reading and learning, in my environment, I set the bandwidth to 20 and the power of the channel to low.  I have 3 AP's in my house.  I read it was recommended to narrow the channel and decrease the power, otherwise there is too much noise the clients have to try and decide which AP to connect to.  Since doing so, I have all excellent wifi connections but more importantly, HA is not dropping the device as unavailable.  I will let this ride and fingers crossed this headache is behind me!!
Thanks for your suggestions!


Bump supermarket cookbook version

Hi,
Would it be possible to upload a new version of this cookbook to the supermarket? There have been a lot of changes and I would rather not pull it directly from the repo.
David

Hey, I believe the title of this Issue is misleading, as you are asking for the version to be bumped rather than a new cookbook to be created.

Yeah, that's what I meant 😄

+1
We really need some bugfixe.
Thx

Done. Finally, v3.0.0 is released and published on Supermarket.
Changelog: https://github.com/johnbellone/consul-cookbook/blob/f2e9e53a487431d10cc2500ed67166ccbf8a5c2a/CHANGELOG.md#v300-2017-06-11


Console erasing/truncating buffer history

From @MaximoTrinidad on May 26, 2017 15:51
This bug-tracker is monitored by developers and other technical types.  We like detail!  So please use this form and tell us, concisely but precisely, what's up.  Please fill out ALL THE FIELDS!
If you have a feature request, please post to the UserVoice.  If you're reporting a BSOD, don't post here!  Instead, e-mail<EMAIL_ADDRESS>and if possible attach the minidump from "C:\Windows\minidump".

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.16199.1000]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
After executing a number of Linux commands, I scroll-up to review what I type. The history displayed is limited and only scrolling one page.

What's wrong / what should be happening instead:
Seems like buffer history it's been erase or truncated so when you scroll-up only a limited number lines are displayed back. It should display all commands with results of what I've done.

See our contributing instructions for assistance.
Copied from original issue: Microsoft/WSL#2161

From @zadjii-msft on May 26, 2017 19:49
You previously ran apt in that console window, right?
This is #1984 IIRC


Provide option to auto-relay

It would be useful to have an option in MailHog config to auto-relay all messages.

Thanks @obrienmd
I'd be interested in understanding your use case? Adding that option wouldn't be too difficult, but MailHog isn't designed to work as a real MTA and doesn't make any SMTP delivery guarantees - the existing implementation (being a manual thing) makes a delivery failure less important.

Thanks for the feedback and question - makes perfect sense.
In some cases as we move a change to a staging environment, we would love to have both the e-mail captured in MailHog (for immediate checks) and allowed through to a group of test end-users (for more delayed but wider feedback). MailDev has a pattern-match option for auto-relay, which would be ideal, but full-bore auto-relay would be perfectly fine (and probably much easier), as we can just manage the users in the staging environment.

To further expand on this, it would be for large groups of users on a staging environment where manual relay via MailHog would be difficult.

Thanks for the info! I'll add something :+1: not 100% sure what yet, but something!

Cheers - honestly, just a boolean config / cli option for "autorelay" would be awesome :)

For me, I'd just like a button that auto-releases. In other words, not for every message (although no reason not to do so), but instead of having to re-type (or copy/paste) the To: field, just have a button that ships the entire message to the defined SMTP server

Hi All,
MailDev has an auto-relay-mode with a json config file to deny or allow the recipient.
Regards

Yep - unfortunately, I'm unable to use the node runtime in this situation due to silly policies :)

Hello,
Such a feature would be indeed nice to have. Even better if it could be turned on/off based on some condition e.g. "auto release all emails for recipients not in domain "example.com".
Regards,

Hello,
We have historically used mailcatcher in our dev and QA environments, but it's not sufficient to handle production pass through (without a feature such as auto-relay).
The use cases are:

We send thousands of emails daily (not spam or ads, these are all opt-in edoc delivery of individuals insurance policy documents).  These emails are generated by pulling data from different middleware, dbs, and 4GL appserver wackiness, and once in a blue moon there are mixups that cause issues.  It would be great to be able to validate when a user calls in by looking in the "pass through catcher" to see what they are seeing (asking a user to forward an email without mangling it can be a chore with our user population).

Helpdesk case #2:  When a user calls in and says that they have not received their one time password that expires in 15min for 2FA because they are on some ISP that spam checks everything so delays mail delivery by that much, the helpdesk can review the email to them and provide the 2FA code.
Helpdesk case #3: When a insured calls in and wants help viewing their inspection or other attached document to their account, the helpdesk can open up what was sent to them and walk them through the exact process.
I understand this would also allow users to capture forgot password kinda emails.  To prevent this kind of abuse,  we currently lockdown access to mailcatcher (on dev and qa as stated above) with https on an internal network and unique named user proxy access only.  The applications that we want to catch email from are not high on the risk level for a third party accessing (some risk from PII disclosure, but you can't do anything financial other than pay someone's policy premium, and we don't save financial creds, so a malicious user would have to pay with their own CC or ACH).  The users on the HD who would have access to view these outbound emails in the pass through have WAY more access with their internal policy modification tools.
Again, thanks for a great tool (demoing now to replace mailcatcher) and +1 to this use case.

Also, I've tested maildev extensively(since as mentioned above, it does provide this) but I've not found it as stable as mailcatcher.

So it sounds like there's two distinct use-cases here, with subtle differences:

To be able to release the message as-is to another mail server - MailHog would send the message to one mail server, and that mail server would be responsible for delivering to all recipients (MailHog is a proxy)

To be able to release the message to its intended recipients - MailHog would need to parse the message, group recipients by MX server, and deliver the message to the correct recipients individually (MailHog is an MTA)

The first one is a possibility - and is easy to add - since MailHog just forwards the raw data as-is to another server.
The second one is unlikely to be supported, since it requires complex mail delivery rules and would be nearer a real MTA at that point.
So I'm happy to add support for the first - though I'm not sure how soon I'll be able to get around to it. But I don't think the complexity etc of the 2 is really worth it, when 1 is an option.
@chines that's impressively outside the scope of what MailHog was designed for, and I'd recommend against using MailHog in a production environment like that even if the auto-relay functionality was available

#1 is the one I was describing, just forwarding the raw message along to another mail server.
Thanks for the suggestion against using it in a production environment, and I hear that loud and clear.   I'm going to test it for a few months and see if it's stable enough.... ;)  The benefit would be pretty high and as long as I have a fairly simple recovery plan for failure scenarios, I think it would be workable.

Option 1 would be fantastic :)

Option 1 is the kind of thing I need. @chines I think in your other situations you want an entirely different approach - I'd probably just configure your tool/existing MTA to BCC every email to a research box or something.

+1 for option one. For my use case it would be enough to prefill the recipient field of release message modal dialog with the original recipient. Nonetheless another action to release to all original recipients (including CC and BCC) would cover far more use cases.

+1

Can this feature also be extended to only allow or deny certain recipient email patterns?

+1 needed feature
MailHog can be not an MTA, it can just forward mails to another SMTP
It is not very hard to write a script that will listen the SSE events with letters from mailhog and forward them via this script, but it would be nice to have this built-in


⚠ Bofrost.de has degraded performance

In d612087, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 934 ms

Resolved: Bofrost.de performance has improved in 3f81c20.


Add an 'Update LKG' action

As far as I can tell, I think it does that: https://github.com/microsoft/TypeScript/runs/943760878?check_suite_focus=true#step:4:139

Yeah, the npm test command is in the yml file

Oh, duh, I missed it because I was looking for the gulp command directly.


Cannot use package

Platform: Google Colab
Version: Python 3.10.12
Issue
I am unable to use the package despite it being installed

As you can see I tried installing the package from pip as well as the release zip file

Oh, thanks for the note! I forgot to actually include picopipe in the package. Should work now:
%pip install picopipe==1.0.2

Fixed with 96ca1d6e2bf113ab3e19e8077839ed10a46e99f4
Thanks! Closing this issue.


Connect mining core thru mining device using miner username instead miner wallet address

Hi @blackmennewstyle  Sir:
     Thank you for your support & cooperation!

Actually Sir I have set up mining core and it is running. In this set up LTC coin is added. To check mine is working  for this coin or not, I have run cpu miner and connected with my mining core. To run the cpu miner I gave stratum url, wallet address for LTC.   And from mining core log, I got this message "Connect from Authorized miner........" . Means it is working.  Now Sir, like some other mining pool to connect mining core If miner wants to put his/her username in the mining device which he/she have received at the time of mining pool registration instead of wallet address then what are changes I need to take care?
Thanks in advance!

Miningcore does not support registration for mining. It uses wallet address as authentication, it has been like that since its creation.
KYC is a bad aspect for crypto mining anyway. Do not go down that road.


executed no results

I installed latest ad module
copied the contents of the script into a text file, renamed .ps1
executed and stays running without any output , after typing credentials.

Could you add some screenshot please?

Please show me a screenshot of your ps1 file.
Couldn't you just download the file from my github repo?


⚠ Bofrost.be has degraded performance

In d23ea00, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2435 ms

Resolved: Bofrost.be performance has improved in 2c36d28.


Add hx-ws

Hey Chase, I didn't add hx-ws in my PR because of this:

But since you already added it, maybe you can also include hx-sse too, which is in the same condition (moved to an extension)

@dreth can you link that? Can't find it. From another page, I see that it only moves to an extension in v2, correct? I don't see an issue with adding support for anything in v1 as well. Is there anything else you thing would be beneficial as well that is currently missing?

https://htmx.org/extensions/server-sent-events/
import:  https://unpkg.com/htmx.org/dist/ext/sse.js

https://htmx.org/extensions/web-sockets/
import: https://unpkg.com/htmx.org/dist/ext/ws.js

Thanks @gedw99 ! Do you all think we should add sse-connect, sse-swap, and ws-connect as well?

What @gedw99 linked is more complete, but the screenshot i attached is from the end of this table: https://htmx.org/reference/#attributes-additional

Sure, that would be awesome. Thanks @dreth !


Easier AccessLogs configuration (ex for HTTP Headers)

My starting point is that I would like to add to access logs single HTTP header (all of them would be too heavy), but currenty there is no "nice" configuration for that.
The current workaround would be to redefine whole Json pattern via AccessEventJsonPatternLayoutEncoder and add my header to it.
My proposals:

provide direct configuration possibility to choose which headers are to be logged
AND/OR
for AccessEventJsonPatternLayoutEncoder (ond other encoders too where it applies) enable "merging", so the pattern I provide is merged into standard one

This would be useful for us also as want to easily exclude certain headers (like Authorization).

Ability to include/exclude request/response headers added in 5.0


Add support for disabling brotli, parquet and protobuf

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
For changelog. Remove if this is non-significant change.
Category (leave one):

Build/Testing/Packaging Improvement

Short description (up to few sentences):
Add support for cmake variables ENABLE_PROTOBUF, ENABLE_PARQUET and ENABLE_BROTLI which allows to enable/disable the above features (same as we can do for librdkafka, mysql, etc)

Building with -DENABLE_PARQUET=0 can be used as workaround for https://github.com/yandex/ClickHouse/issues/4658 in case you don't need parquet.


Add consistent inactive zones around panes

:clipboard: Steps

Hover near the panes to see how countries are selected on mouse hover.

:bug: Actual result
Only one inactive zone is present at the moment - the .AreaList::after pseudo-element.

:mag: Expected result
Current inactive zone should be expanded to cover the left side of the .AreaList (in fact, the entire "bottom-left" corner occupied by the .AreaList). Another one should cover the "top-right" sides of the . AreaInfo.

43c14d35d36f1332c837890371b59ecde1ca1ab7


youtube-dl: 2020.11.12 -> 2020.11.19

https://github.com/ytdl-org/youtube-dl/releases/tag/2020.11.18
https://github.com/ytdl-org/youtube-dl/releases/tag/2020.11.19

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks! Ported to stable as ae7f176f4f96e5058ab50d738ab69cad73a07c4f.


Make Battery Power signed for Growatt SPF

Probably a typo, makes negative values appear as negative.

Description
Fixed signed value for BattPwr
How Has This Been Tested?

[ ] Not applicable

Inverter type

[ ] Simulated inverter
[x] Growatt SPF 6000 Plus

Stick type

[ ] Shine X
[ ] Shine S
[ x] Lolin32
[ ] Nodemcu32

@leodesigner Thank you for your contribution. Could you signoff your commit and fix the format check?

Could we merge this?

No there has been no signoff for the orginal commit. @leodesigner  see here how to fix this: https://docs.pi-hole.net/guides/github/how-to-signoff/


Requirements say only PHP 5.3+ is required, but PHP 5.4 or possibly higher is actually required.

Was trying to run this on a server with PHP 5.3.5, since the requirements on this page say PHP 5.3+ is required. I kept getting an internal server error when trying to run the example code. After poking around in the internals, I noticed that one of the files was using short hand array instantiation:
$var = [];
Short hand array instantiation was not introduced to PHP until version 5.4, and that is why I was getting an internal server error while running the example code. I am not sure if there are other features used in the SDK that would push the PHP version requirement even higher, but I can say for sure that at least version 5.4 or higher is required.

@dqhendricks Thanks for pointing it out. We have updated the README accordingly.


Create gui on rewrite

The failed test will work once merged. This is because the scripts directory is not cloned when there is no file in it. The master branch has files in it.

This works like the current yapi gui. The 0 in the result is the exit code of the process for installing. I need to figure out how to intercept terminal output instead of the exit code which is returned by subprocess.call()


Should we check single export against the collector peer as peering

If the AS Path is a single AS, we bail its peering check:
https://github.com/SichangHe/internet_route_verification/blob/a1b1b2aab751688d3524d178cfd70bb3dae6c7a6/route_verification/bgp/src/cmp/compliance.rs#L48
However, to my understanding, the AS should be exporting to our collector peer (also referred to as vantage point?). Is that correct, @cunha? So, should we add prepend the collector peer to the AS Path (simple) or give it special treatment (complicated)?

Completely ignore them.

The export to the route collector may be a special rules not added to the IRR.


Add built-in item building

Goals

To make the item building significantly easier while allowing for extra modification and extensibility via an abstract ItemBuilder

Example
Button button = ItemBuilder.simple(Material.ARROW, 1)
                                             .name(Component.text(...))
                                             .enchant(...)
                                             .allItemFlags()
                                             .clickable((menu, event) -> ((Player) event.getWhoClicked()).sendMessage("I was clicked! at slot " + event.getSlot());

// do what you want with "button"

A lot of unnecessary things -> rejected !


Allow custom fields to be added to resume key

Type of issue

[x] Bug report
[ ] Feature request
[ ] Support request

Uploader type

[ ] Traditional
[x] S3
[ ] Azure

Bug Report
Fine Uploader version
5.14.0
Browsers where the bug is reproducible
any
Operating systems where the bug is reproducible
any
Exact steps required to reproduce the issue

Upload part of a large file to S3 with metadata: {user: 123, project: "xyz"}
Refresh page
Select same file to upload, but with different meta data: {user: 456, project: "abc"}  (actually, I don't think this is possible, but a different file would be given this metadata)
File upload continues from where it was interrupted
When file is complete, metadata is as per original upload

Detailed explanation of the problem
Resumable uploads are saved to localStorage with fileName, size, uuid, modifiedDate etc, but metadata is not included.  If a different user logs in to our app, or the same user switches to a different project and resumes the upload then when complete, the file will belong to the original user/project.
The code that parses the resume data from localStorage should compare the metadata, or onResume should provide some way for the application to compare the metadata (request.params).

I see your point, but we'll need to define "metadata" exactly first, in the context of Fine Uploader S3. And second, we'll need to make appropriate changes to the code that serializes data to localStorage. I'm a bit apprehensive about that second step, as I don't want to run the risk of running out of space, since localStorage space is limited to about 5 MB or so.

Also, this is more of a feature request than a bug. Fine Uploader S3 functions as designed here.

Fine Uploader allows for an optional additional request.params object to be configured. For S3 uploads these are implemented as metadata fields on each S3 (file) object.
I don't imagine that storing the request params would add much overhead.  While it may bring forward the quota exceeded scenario, that possibility of running out of space was always there. Perhaps the resume feature needs configurability of max age and record count?

Age of records is already a configuration point. Another tricky implementation detail: ensuring this does not break existing/persisted resumable files.

Sorry, decided to not implement

I'm finding a possible need to the ability to store additional data in resume record keys. So I'm going to re-open this issue.

Note that this should be addressed in #1916.


feat:upgrade go version and solve unreasonable package dependencies

Description
Please include a summary of the changes.
upgrade go version and solve unreasonable package dependencies
This bug fixes ... \ This new feature can be used to ...
Context
Users ran into this bug doing ... \ Users needed this feature to ...
Checklist:

[ ] I included a concise, user-facing changelog (for details, see https://github.com/solo-io/go-utils/tree/master/changelogutils) which references the issue that is resolved.
[ ] If I updated APIs (our protos) or helm values, I ran make install-deps operator-gen generated-code from tools/wasme/cli to ensure there will be no code diff
[ ] I followed guidelines laid out in the Gloo contribution guide
[ ] I opened a draft PR or added the work in progress label if my PR is not ready for review
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

LGTM, can you also bump the go version used in the build & release logic in the GitHub actions so that we don't run into trouble when we do a release:
2 instances here to change 1.14 -> 1.15 - https://github.com/solo-io/wasm/blob/master/.github/workflows/release.yaml#L16-L19
Also codegen here - https://github.com/solo-io/wasm/blob/master/.github/workflows/pull_request.yaml#L21

done

/retest


[alpha8] Nature's Scythe don't destroy anything without modifires

It does need to be documented though


BUG: launch method exception

Describe the bug
The launch method of XInference encounters an error when loading embeddings, images, and reranking because it does not account for the scenario where size_in_billions is None.
To Reproduce
$ xinference launch --model-name bge-reranker-large --model-type rerank -u af828dd0-8e0a-1101-93cf-437b86dbc17b
Traceback (most recent call last):
File "/home/auxpd/miniconda3/envs/xinference/bin/xinference", line 8, in
sys.exit(cli())
File "/home/auxpd/.local/lib/python3.10/site-packages/click/core.py", line 1157, in call
return self.main(*args, **kwargs)
File "/home/auxpd/.local/lib/python3.10/site-packages/click/core.py", line 1078, in main
rv = self.invoke(ctx)
File "/home/auxpd/.local/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
return process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/auxpd/.local/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/auxpd/.local/lib/python3.10/site-packages/click/core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "/home/auxpd/miniconda3/envs/xinference/lib/python3.10/site-packages/xinference/deploy/cmdline.py", line 501, in model_launch
size_in_billions if "" in size_in_billions else int(size_in_billions)
TypeError: argument of type 'NoneType' is not iterable
Expected behavior
successful loading

I'm a bit curious, why is there a need to perform type conversion here?

@auxpd . Thanks for reporting. This is a bug for registering non-LLM models. We will fix for the next release. For now, you can specify any value for this option.


How to use navigationType ?

Hello thanks for nice plugin!
Any code example that I able to use navigationType ?
it did not show dots or bars
thanks

Thank you for your feedback @kristoff2016 and sorry for my late response. If I understand well you're not seeing any dots or bars using navigationType option?
Do you have some code and/or screenshot for this issue? I'll look into it.

Hi,
Me too I don't see any navigationType.

This is my render method:
  render() {
    return (
      <ScrollView>
        <SafeAreaView forceInset={{ top: 'always' }}>
          <Carousel data={dataCarousel} navigationType="bars"/>
        </SafeAreaView>
        <StatusBar barStyle="default" />
      </ScrollView>
    )
  }

Hello, use this method:
<SwipeableParallaxCarousel
data={this.state.dataSource2}
navigationType={'bars'}
navigationColor='#6c6c6c'
navigation='true'
/>


fix: upgrade version

Closes #3197
Closes #2930
Closes #3206

📝 Description

Add a brief description

⛳ Current behavior (updates)

Please describe the current behavior that you are modifying

🚀 New behavior

Please describe the behavior or changes this PR adds

💣 Is this a breaking change (Yes/No):

📝 Additional Information

Summary by CodeRabbit

Dependencies

Updated @react-aria/selection to version 3.18.1 in multiple packages.
Updated @react-stately/tree to version 3.8.1 in several components.
Updated @react-stately/list to version 3.10.5 in relevant packages.

Bug Fixes

Introduced patches for various Next UI components to fix link errors.

will be handled in #3240


feat(website): prism theme for syntax highlights

This is an adaptation of https://github.com/PrismJS/prism-themes/blob/master/themes/prism-synthwave84.css but made bluer and a bit less wild. Still a bit neon. Probably could user further tweaking.

@roguesherlock Can you review this please?

Updated to this Nord theme

could user further tweaking.


MG::FileUtils

Add MG::File with MG::File.read to get the content of a file from resources directory

Thanks for the patch :-)

Thanks for the patch :-)


Correct typo in docs on acquisition functions

There was a small typo.

fyi I

cleaned the commits, for some reason commits from another unrelated PR were in here
removed your changelog entry, the changelog does not need entries for simple typo fixes
capitalized commit messages

Thank you. I tried to do rebase as it was needed due to changelog, but it seems I messed it up?
@AVHopp the failing test is due to scikit fingerprints right?

@Hrovatin , yes it seems like the tests still fail due to the fingerprints error. We could merge the corresponding PR first, then we should be able t verify this.

@Hrovatin please analyze what happened with the rebase so it doesnt happen again, seems to me maybe it was maybe rebased onto the wrong branch or so.
In any case, be sure to have a backup branch before you do the rebase and let us know if you have any questions / uncertainty around rebasing.


param defaults in script docs

It would help to have the parameter defaults in the script docs. Example code to automatically add defaults:
class CustomFormatter(argparse.ArgumentDefaultsHelpFormatter,
                      argparse.RawDescriptionHelpFormatter):
    pass

parser = argparse.ArgumentParser(description=desc, epilog=epi,
                                 formatter_class=CustomFormatter)

Thank you very much for this!  This will definitely make things easier for users. I have implemented these changes and they will be in the next version 3.1.0


Copy template between zones fails with md5 mismatch

ISSUE TYPE

Bug Report

COMPONENT NAME

SSVM

CLOUDSTACK VERSION

4.11.2

CONFIGURATION

1 region, 2 zones, template registered/exists in zone1
OS / ENVIRONMENT

Issue confirmed on both KVM and VMware as Hypervisors
SUMMARY

When copying existing template from Zone1 to Zone2 (copyTemplate API call), md5 checksum between source and destination (copied/downloaded to Zone2) template fails
STEPS TO REPRODUCE

Register a template in Zone1, wait until Ready, thy to copy to Zone2 via GUI/API

EXPECTED RESULTS

Template copy works (did work in some of 4.5 or 4.6)

ACTUAL RESULTS

Copy operation is shown as success  (in GUI at least), but actually fails with errors (visible in GUI), from SSVM cloud.log "Failed post download script: checksum "{MD5}f916328a9273c511b36e7062a1b12262" didn't match the given value, "{MD5}a92d0b7882f9c3660bb6536206f1ee2a""
Further analysis, show that we are comparing destination md5 with a wrong source md5 - check the image https://pasteboard.co/I2eJYPb4.png or in other words:
In case of VMware env,  when we register the template with Zone1, there are 2 OVA files created on Secondary Storage in Zone1  i.e.
f960f054-2be3-3431-b876-72b2814fdeb3.ova
and
202-2-cac71084-dacc-35e8-9404-f72660bec1c3.ova
(both identical sizes)
Checking cloud.log in SSVM in Zone2 during copyTemplate API call, we can see we are copying over 202-2-cac71084-dacc-35e8-9404-f72660bec1c3.ova file to Zone2 – BUT we are then comparing md5sum (of downloaded file) to the wrong source file f960f054-2be3-3431-b876-72b2814fdeb3.ova.
I did not capture logs from KVM env, but as far as I recall, it failed with same error message.

@shwstppr can you pick this one?

here are two different checksums:

One for the file to be downloaded to compare against its origin

One for checking internal copying

In the case of downloading a regular image these are the same, but when downloading some sort of archive, the archive will be unpacked on arrival and the result will not match the initial checksum. The checksum can not be updated as it may be used for a re-download.
the original design has not taken this into account.
I propose to implement the following:

create an extra field: 'original_checksum'
when the template is to be exposed from a store by copying or downloading, the file is checked against the checksum
if it doesn't match and 'original_checksum' is null
-+ checksum is copied to 'original_checksum' and a new checksum is stored
if it doesn't match and checksum is not null
-+ error

There is a caveat to this, if there is multiple copies around and they don't all have the same checksum..?
@andrijapanicsb any thoughts?


Add support for multiple webpack chunks in iframe

This is a replacement PR for https://github.com/storybooks/storybook/pull/677
This allows an arbitrary number of files to be loaded into the iframe,
not just the single preview.bundle.js file.
Webpack plugins can output chunks in a separate file. In dev, since
file names are predictable, I can add the js file into a head.html file.
When building storybook statically, however, js files have a hash added
to them, so this doesn't work.
This change makes the static build parse all webpack chunks and load
them in the iframe (excluding the manager script), instead of only
including known files.

@apexskier Can you please review and test?

I've extended the original PR to also work for devmode.
The message "storybook started" has been moved till after webpack has compiled.
I've added a progress indicator whilst webpack is busy.

Can someone please review?


Fix tooltip transition

This PR only transitions the opacity property. I think currently it defaults to all and therefore also transitions top and left. Should fix this: https://discuss.atom.io/t/isotope-ui-disable-tooltip-animation/15072 He mentions animations in general, but I'm pretty sure he means the "movement".
Sorry, that got carried over from One and I fixed it only after you forked. :innocent:

I never saw it as a bug, just a bit heavy handed. Thanks for the fix though!


Fix Bukkit scheduler being used in API methods

Using the API to re-render specific towns was failing on Folia due to the bukkit scheduler being used.

Thanks for catching this! I think I'm going to go with a slightly different approach of exposing and using the abstracted scheduler directly, so I'm going to go ahead and close the PR. But I definitely didn't catch the issue before you made a PR, so thanks again!


Docs

[ ] Switch to Jupyterbook (?)
[ ] Execute notebooks as part of the build, fix errors and warnings
[ ] Check for remaining outdated issues

Why would we switch to jupyterbook? If you invest time I would align the documentation infrastructure with optimagic.

Because I am using it all the time and it would free up space in my brain occupied by a mental model of Sphinx, once JB 2.0 has arrived in a stable fashion. Definitely not on my immediate horizon, though.


Delay on save/execute

Loving Cisar.. best thing since sliced bread
With serious editting the constant saving (on every keypress) can cause some issues;
maybe a delay .. saving 500 miliseconds after the last keypress .. can help
This might also prevent the need for having to type a space to update the code?
I am now used to start a line with // (comments) and then type the complete code in that line, then removing the comments.
I presume that is why you added the Ctrl-K shortcut I spotted in the source code.

Thank you soo much for your feedback! :)
It actually has quite significant delay, 1 second or even more. I think I added it very long ago. Are you using version from Chrome Web Store?

I mean, delay is there for save, but not for execute. Execute happens instantly. If there's an error, there's a reddish background, which I personally find very convenient, because I can see it out of the corner of the eye, so I instantly know if there's still error in the code or not.

Hi Danny,
Could you explain a bit in more detail what you mean? Or is it already fixed for you and we should close this?

This is an open issue.
I haven't found the exact cause but every now and then I write code that completely stalls Cisar and the browser, and I have to close the tab.
I think it has something to do with typing opening (round) brackets.  (so suspect a regex involved)
Maybe an option (Ctrl + ??) to only do a live update on a newline or ctrl-Enter

Live refresh could be an option, yes, but I still need to fix the bug too. I think I now understand what you mean. I also experienced couple of times that the browser freezes completely and Chrome Task manager has to be used to kill the tab process, but I thought it was because I'm running extension in Developer mode and often debug the extension itself.
But ok, I'll try reproducing this then. If you get more information of how you can reproduce this bug reliably, please inform me right away.


Break up Augmentations.py, comply with PEP8 & PEP420,

Another big change in the refactoring effort!
In this PR I did the following:

Broke up Augmentations.py into separate modules for each class, all of which now live in the base directory. paperfactory.py now lives here too. This reflects the role of these modules as core components of the whole project.
Separated modules in augmentations (everything other than PaperFactory that inherits the base Augmentation class) into digital and physical directories based on the kind of augmentation being simulated. For example, low ink augmentations went to physical, while the JPEG compression augmentation went to digital. This should help discoverability and is part of the future effort to make a set of primitive augmentations from which others can be composed.
Fixed module imports to conform to the PEP420 standard for implicit namespaces. This brings the naming and import scheme into line with the rest of the ecosystem since Python 3.3. This also allows us to remove the __init__.py files, which I've done.*
Renamed src/Augraphy to src/augraphy, to better conform to the PEP8 standard on package and module names. Same motivation as (3) above.
I factored the probability calculation in OneOf's constructor out into another method. I plan to do this pretty much everywhere, to improve readability and maintainability.
Removed some undeclared shadow dependencies on other modules. For example, I removed add_transparency_line from the AugmentationSequence class per issue #16.
Changed all comments to the Sphinx docstring format so we can generate much of the package documentation.
Added comments to a few more methods.

The initialization code that defined the default Augraphy pipeline still needs to be dealt with. We can probably spread this across documentation examples and tests.

Overall, this is another great PR with lots of improvements to the code quality as a result.
The one area that I think we may want to revisit is the breaking up the digital and physical directories.  Taxonomies are notoriously hard to get right, and I think this is one of those situations where the distinction will not always be so straightfoward.  This may again touch on the notion of building a framework that allows for optimal composability as discussed in #20.


Ajax call blocking application start if called from Program.Main

Hello,
I think it's a racing condition but it seem that AJAX call cannot be done at app start from Main C# context.
My app fail to load on my tested browsers, Chrome and Internet Explorer 11 (i added some extra .js file in my project order to start Blazor on it at the moment).
Considering this kind of example:
namespace MyApp
{
    public class Program
    {
        private static async Task CachingData()
        {
            var resourceHelper = ServiceLocator.GetInstance<IResourceHelper>();
            if (resourceHelper == null)
            {
                JConsole.Log("ERROR: ResourceHelper is not set!!!");
                return;
            }

            await resourceHelper.InitializeAsync(); //DOING SOMETHING AJAX HERE WITH System.Net.HttpClient
        }

        private static BrowserRenderer br;
        private static IServiceProvider serviceProvider;

        public static async Task OnReady()
        {
            br = new BrowserRenderer(serviceProvider);

            await CachingData();

            br.AddComponent<App>("app");
            return;
        }

        static async Task Main(string[] args)
        {
            serviceProvider = AppStart.ConfigureServices(); //SOME SERVICE PROVIDER CONFIGURATION
            JConsole.Log("Services are configured...");

	    await OnReady();
        }
    }
}

My browsers will hang instead of doing the AJAX call before adding App component.
As a workaround, i'm calling the related code in a delayed context, with my own SetTimeout implementation (using under the hood System.Timers) in order to release the current starting thread without actually creating the App component.
Code should look like this:
namespace MyApp
{
    public class Program
    {
        private static async Task CachingData()
        {
            var resourceHelper = ServiceLocator.GetInstance<IResourceHelper>();
            if (resourceHelper == null)
            {
                JConsole.Log("ERROR: ResourceHelper is not set!!!");
                return;
            }

            await resourceHelper.InitializeAsync(); //DOING SOMETHING AJAX HERE WITH System.Net.HttpClient
        }

        private static BrowserRenderer br;
        private static IServiceProvider serviceProvider;

        public static async Task OnReady()
        {
            br = new BrowserRenderer(serviceProvider);

            await CachingData();

            br.AddComponent<App>("app");
            return;
        }

        static void Main(string[] args)
        {
            serviceProvider = AppStart.ConfigureServices(); //SOME SERVICE PROVIDER CONFIGURATION
            JConsole.Log("Services are configured...");

	    Browser.SetTimeout(async delegate ()
            {
                await OnReady();
            }, 10);
        }
    }
}

I don't know if it's normal or a bug, but from this context, i'm not aware that something is not fully "ready", or maybe it's just the Main entry point not being able to start as an async Task Main, failing at asynchronicity even if it compile.
I can confirm that this bug occur with and without browser-polyfill.min.js and fetch.js (intended for launching Blazor in IE11)
Also, a more minor bug on Internet Explorer 11, when doing some AJAX request in this startup context, meaning without App component rendered yet, it seem that if i do any action needing AJAX, any line of code is then called and executed but AddComponents fail to actually render the component. I end with the default page, as the "app" tag is not loaded at all.
If comment the AJAX request everything work fine.
As a workaround, i actually done a IHttpClient interface in my app to wrap http calls, and actually using jQuery instead for some ajax calls, if the current browser is Internet Explorer. Doing my calls from the $.ajax context of jQuery do not create this AddComponent bug at app start, and everything work like the expected behavior.
I know that Internet Explorer 11 is not yet officially supported on your side, and maybe it's something related to the missing libraries (promise etc.).

I'm not sure what's wrong in your code, but I just tried the following in the StandaloneApp code and it worked fine:
        public static async Task Main(string[] args)
        {
            var serviceProvider = new BrowserServiceProvider(configure =>
            {
                // Add any custom services here
            });

            var httpClient = (HttpClient)serviceProvider.GetService(typeof(HttpClient));
            var result = await httpClient.GetStringAsync("/sample-data/weather.json");
            Console.WriteLine(result);

            new BrowserRenderer(serviceProvider).AddComponent<App>("app");
        }

If you still believe there's a general bug with HTTP calls from Program::Main, could you provide more targeted repro steps that don't include other custom library code (such as IResourceHelper in your sample above)?

Hi Steve,
I can confirm, this bug does not occur anymore, by testing the exact same scenario. Maybe some nightly evolutions on your side resolved this strange issue since i have updated Blazor.
Thanks Steve, sorry for the inconveniance.


no-objc patch for all supported branch specs

Looks like master branch failure not related to these changes


Capture more macos shortcuts

Found different behaviors between using arrow keys and MacOS's native shortcuts (like Ctrl-f) to move cursor.
Adding some conditions in captureKeyDown will solve this issue.

Thanks!

Well, on closer look, this patch was broken and apparently never tested before being submitted. Attached patch cleans it up.

Sorry. I perhaps copied the code from old one...


Update base.py

I received an error that the module collections did not have 'iterable'. in the last version of the code, indeed this was the case. Hopeful this may prevent import issues of others as well

Implementation which is now fixed for me:
def get_posts(site_url, username, password):
wp = Client(
url=site_url,
username=username,
password=password,
blog_id=0,
transport=None
)
posts = wp.call(
method=GetPosts()
)
for post in posts:
print(post.title)


Captcha

Proposed changes
Implements CAPTCHA functionality
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] DevOps related

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the project guidelines doc
[X] I locally linted my code
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
Implemented CAPTCHA functionality. This CAPTCHA is Google reCAPTCHA Invisible v2, meaning that it will only show the CAPTCHA check if it thinks it is a bot, otherwise it will not inconvenience the user.

Hiding CAPTCHA badge is an option, see image

@CollinsMathews You can go ahead and merge whenever you're ready


Update README

Codecov Report
Patch and project coverage have no change.

Comparison is base (75ac554) 53.46% compared to head (2aced3b) 53.46%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #321   +/-   ##
=======================================
  Coverage   53.46%   53.46%
=======================================
  Files           2        2
  Lines         101      101
  Branches       25       25
=======================================
  Hits           54       54
  Misses         42       42
  Partials        5        5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Registering task with non-service account not working

Usage of Register-ArmaServerTask with -UserId xxx is not working.
For details, see: https://stackoverflow.com/a/70793765/6092856

We should use different principals for the scheduled task and running the game.
The scheduled task needs write access over steamcmd for autoupdate and allow login as batch job.
This could run under the current user's principal unless another account is specified (it should not be LocalService).
It also needs:

Read access to the server configuration file and the STEAM_USERNAME environment variable
Write access to workshop content, master, config and transcript logs

The game should run as LocalService with as few privileges as possible. It needs:

Read access to workshop content, master, config
Write acess to pid files and profile directory


docs(bidirectionality): bidirectionality how-to explanation

fixes #4387

@EladBezalel Can you please elaborate or prepare a plnkr on how to change the direction of the document from the code, since when I import Directionality out of @angular/cdk and inject it into the constructor of the app component, I can read the value, but when I set it to "rtl" nothing happens, not even the change is being emitted.


Applications and draft not showing when I click begin processing applications

When I click begin processing apps on staging the applications section then does not appear when I know there are draft and sent applications.

We have re-factored the logic which determines when the Applications side navigation displays... So hopefully this bug has been resolved as a result.


Add a number of docs from Canton into this repo to unblock current HA and Scaling work

I am currently blocked on HA and Scaling documentation due to problems that arose when I was trying to delete a doc from the Canton snapshot to remove errors in the build.
For reference: https://github.com/digital-asset/docs.daml.com/pull/138
@garyverhaegen-da suggested that to solve this we could remove the whole section from Canton and add it directly here.
The docs that need to come into the main branch are the rst files for:
https://docs.daml.com/canton/architecture/ha.html
https://docs.daml.com/canton/architecture/ha/overview.html
https://docs.daml.com/canton/architecture/ha/participant.html
https://docs.daml.com/canton/architecture/ha/domain.html
They also need to be removed from the Canton repo.

any idea on timeline on this one @garyverhaegen-da? I can't do this.

I should be able to get to this early next week.

@da-katmurp I believe this is resolved. If not, could you please point out what is missing?

Yes all good now.


update

Here are some reminders before you submit the pull request

[ ] Add tests for the change
[ ] Document changes
[ ] Communicate in the mailing list if needed
[ ] Pass make installcheck

update from main


TEN-105 Adoption Dashboard component

Pull Request Checklist

[ ] Target branch: Pull requests should target the dev branch.
[ ] Description: Briefly describe the changes in this pull request.
[ ] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[ ] Testing: Have you written and run sufficient tests for the changes?
[ ] Code Review: Have you self-reviewed your code and addressed any coding standard issues?

Description
[Insert a brief description of the changes made in this pull request, including any relevant motivation and impact.]

Changelog Entry
Added

[List any new features, functionalities, or additions]

Fixed

[List any fixes, corrections, or bug fixes]

Changed

[List any changes, updates, refactorings, or optimizations]

Removed

[List any removed features, files, or deprecated functionalities]

Security

[List any new or updated security-related changes, including vulnerability fixes]

Breaking Changes

[List any breaking changes affecting compatibility or functionality]

Additional Information

[Insert any additional context, notes, or explanations for the changes]

[Reference any related issues, commits, or other relevant information]

Looking great, can be merged.


Upgrade to 0.28

As of right now, everything looks great, except switching out 0.27.2 to 0.28.0 causes a strange error:

The stack trace is blow away.  Here's our starting point.   https://github.com/facebook/react-native/blob/516bf7bd94d14600920a3d78fdcf51ea7fe48495/Libraries/EventEmitter/NativeEventEmitter.js#L32
@skellock @kevinvangelder @fvonhoven  - Would send mayonnaise to first person who solves this.

take a look at this:  http://stackoverflow.com/questions/37999468/entry-file-error-in-react-native-from-packager
seems to be the same error that packager is saying.

I've narrowed this down to push notifications.  Commenting out pushconfig fixes:

simply adding
import PushNotification from 'react-native-push-notification' causes this error.   Will file with the npm module source.   For now I'm likely to disable push in ignite.   Keeping this lib working is a bit of a burden.

Solved by removing push #210

Can you post your package.json file? This situation seems to happen when we are having a lower version of rn plugins?


Move TP-Link socket LED state setting to update()

Description:
LED state setting will still only occur on the first update() then the device is available.
Handles errors when setting the LED state of TP-Link sockets.
Example entry for configuration.yaml (if applicable):
- platform: tplink
  host: xxx.xxx.xxx.xxx
  enable_leds: False

Checklist:

[x] The code change is tested and works locally.

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Looks fine for me, travis-ci errors weirdly enough on non-relevant code. Any ideas? Maybe rebasing this on top of the newer master could help?


Add nonce and soap:actor support for WSSecurity

Coverage increased (+0.1%) to 92.406% when pulling 18db0e30c30715b44a01b5d117753160a0957d8e on strongloop-forks:master into 448c903a68e4fd88926f346677e565dbdcd48f75 on vpulim:master.

Coverage increased (+0.1%) to 92.406% when pulling 290f869d5dced6fd4e672f573bb5ad22c9e46aa5 on strongloop-forks:master into 448c903a68e4fd88926f346677e565dbdcd48f75 on vpulim:master.

@vpulim @jsdevel Any chance to review and merge the PR? We would love to have a release with this fix soon.

LGTM.  Thoughts @herom ?

Coverage increased (+0.2%) to 92.41% when pulling c27d54d49c160fc38a4b8d40a97300aac520bef6 on strongloop-forks:master into 448c903a68e4fd88926f346677e565dbdcd48f75 on vpulim:master.

@jsdevel Thanks! I look forward to a new release in npm soon.

@herom @jsdevel Would you please publish a new version to npm? We're blocked.

I'll try to get to this today.

@raymondfeng sorry for the delay.  This was published with v0.16.0.


MGMT-13065: Add expired certificate signature

In case the signature is matched it will add the following comment:
h1. Expired Certificate
{code}x509: certificate has expired or is not yet valid: current time 2022-12-23T10:25:09Z is before 2022-12-23T15:18:24Z, verifying certificate SN=4206087047961838051, SKID=, AKID=D9:86:D3:E7:6A:99:3A:0B:05:F3:C3:EB:FD:E3:5B:41:4F:EC:32:18 failed: x509: certificate has expired or is not yet valid: current time 2022-12-23T10:25:09Z is before 2022-12-23T15:18:24Z]"{code}
 additional 1016 relevant similar error log lines are found