[SMALLFIX] Clean up ufs journal gc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15892/
Test PASSed.


fix(ci): pass SDK_BRANCH and BUILD_NUMBER to FSC

@msohailhussain is this still relevant? Should it be merged in?

yup, it's still relevant.


docs: add links to top with jump2header

Added links to the table of contents for better navigation.
Demo
Generated with:
jump2header --slug "table-of-contents" --start "Youtube" --end "Directory Tree" --maxLevel 2

If you don't like the PR but interested in links to top see jump2header for more options.

邮件已收到，我会尽快回复。

来信收到，谢谢，祝您一切顺利！

您好，您的邮件我已收到！祝工作顺利，天天开心。。。


Just write: Minions waiting...

That's all i see. No errors in console and no bots. What's wrong?

Could you please provide me with a picture ?

thanks, but i deleted ragapw chrome extension and get it works


Add prefix to MissingTrieNode exception

What was wrong?
Some future requirements are to:

get the prefix of the key used to navigate down to a trie node that then raises a MissingTrieNode exception
traverse the trie directly to a node, using a series of nibbles

How was it fixed?
Requires traversing down the nodes in a different way.
Includes another breaking change: Drop KeyError as a superclass of MissingTrieNode.
(This is an extraction from #95 )
TODO:

[x] rebase on #97 to drop py3.5 support (PR uses f-strings)
[x] ~maybe merge with the other trie iter utility~ (After adding TrieFog)
[x] add release note about breaking changes

Cute Animal Picture

This is a step toward improving the trie iterator. After #95 then I should be able to simplify (and improve performance) of NodeIterator.


Introduce a time-aware tracer

...instead of relying on new ticks to print the time. The upside is that we now print the time of events that occur mid-tick (and not at the beginning of ticks), which can be useful when debugging eg. timeout-based tests. The downside is that ticks that happen at the same absolute time are not separated anymore (although maybe that's for the best?) and we rely on IOSim a bit more because we assume that time moves in chunks.

My intuition, so far, is that time is only advanced between ticks. What is the purpose of printing the time for every trace message?

We don't print the time at each message, we only print a horizontal bar and
the time when the time changes (which makes sense in IOSim but not in IO).
The thing is:

Time may remain unchanged between two ticks (in case of zero-duration
ticks).

Time may change within a tick (in case of timeouts or LoP or other
time-based mechanisms).

I found it frustrating to not have the time of events that happen not at
the beginning of ticks and wanted them to stand out a bit more. I feel this
is achieved. However, I am not particularly convinced by the fact that the
traces would become awful in IO or by the fact that ticks aren't
necessarily separated anymore.
I think something that could make sense would be to always separate ticks
with horizontal lines and to prefix the messages by their time if they
don't happen at the beginning of a tick? This would end up prefixing all
the messages in IO and only some in IOSim, but it would achieve my initial
goal without being as disruptive as the current work.
In general, I think we're hitting something that's been puzzling me a bit
for the past weeks: we basically have two semantics of ticks:

The intended semantics is that they are blocks of time long enough to
ensure that the system stabilises with the new state before the end of it.

The IOSim semantics is that the system stabilises instantaneously and
that we then wait for the duration of the tick.

These are close but behave a bit differently with zero-duration ticks or
with time-based events unrelated to the schedules. I think we rely more on
the IOSim semantics in many places and some tests might therefore not even
pass in IO.
On Mon, 11 Mar 2024, 10:51 Facundo Domínguez, @.***>
wrote:

My intuition, so far, is that time is only advanced between ticks. What is
the purpose of printing the time for every trace message?
—
Reply to this email directly, view it on GitHub
https://github.com/IntersectMBO/ouroboros-consensus/pull/990#issuecomment-1988155124,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNFOWRHZTQQKWMS66MPJG3YXWEE5AVCNFSM6AAAAABEP5SUKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYGE2TKMJSGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Time may remain unchanged between two ticks (in case of zero-duration ticks).

Time may change within a tick (in case of timeouts or LoP or other time-based mechanisms).

These looks to me like good reasons.

some tests might therefore not even pass in IO.

Our simulated peers wouldn't work in IO because of how running the schedule dispatches the state updates. I think it would be fine to continue to disregard IO.

I think something that could make sense would be to always separate ticks with horizontal lines

When reading traces, it has been helpful to jump between the point schedule and the traces specific to a tick. It would be worth keeping this navigation easy to do. Perhaps the horizontal lines with separators work for that sake.
Also maybe share here an example of a trace?

pretty cool

Hum, I enjoy the alignment but we now have a prefix of 38 characters + QuickCheck prefix. Most of the time, 10 of these are useless (because most of the time things happen at the beginning of a tick) and often some others are (like there is a loooot of padding after “ChainDB”). I think that's why we had initially given up on the alignment and systematically prefixing with time. I'm also not convinced by dropping the “Scheduled”. It is obvious for us, but I wouldn't want to confuse a random person trying to debug why a test broke in the future. Still, that's all a matter of taste, so let's keep it that way.


Alternative implementation of extension support

I've made my own thoughts on the extension support and this is what I came up with!
The main difference is that there is no proper abstraction of extensions: An extension is simply a trait that has some special symbols that are used to export the interface with a extension_data function. Both the "special symbols" and a extension_data implementation can be generated with a macro. My reason for not having a proper abstraction of extensions is that you can not really abstract traits:
You can only implement a trait for a proper type or a trait object, not for a trait itself (meaning that the implemented methods are simply added to the trait). Trying to circumnavigate these shortcomings doesn't really lead to a nice, somewhat idiomatic solution and therefore, I used the simplest one!
The resulting code is short, relatively easy to understand and it works. I therefore hope that you like my approach!

I'm sorry, creating this pull request wasn't constructive.


Add support for blake2s hash function with 256-bit output

See crypto/src/hash/blake for an example implementation of blake3.

Ready for review here 8fa2140


dll name in DllMain.def is different with the actually file name when built with cmake on Windows

dll name in DllMain.def is "StormLib.dll", but when built with cmake on Windows, the actually dll file is named "storm.dll".
I'd suggest remove the dll name ( which is optional ) in DllMain.def.

I removed it from the DEF file (https://github.com/ladislav-zezula/StormLib/commit/910b9ef2c1b0e563a20505aad59b9c44aaf8785d). However, the name should rather be StormLib.dll to avoid possible conflict with Blizzard's storm.dll.

However, the name should rather be StormLib.dll to avoid possible conflict with Blizzard's storm.dll.

So, maybe
if(Win32)
set_target_properties(${LIBRARY_NAME} PROPERTIES OUTPUT_NAME "StormLib")
endif()

in CMakeLists.txt (after the add_library line) is more appropriate?

That doesn't work. I guess that's one of the reasons why I hate CMake to the bone. If you can, please, make pull request.

It should be if(WIN32) instead of  if(Win32)


Admit (Selected) from Waiting Room removes Selections upon completion

When sending (selected) participants to the waiting room the selections remain selected upon completion. Although, when admitting (selected) participants from the waiting room the selections or no longer actively selected upon completion.
The initial selections just should remain selected in the workflow.

This may not be technically possible since the selected users are given new Zoom IDs when they enter the meeting. So the people you selected actually go offline and new people with the same name enter. The selected ZoomIDs would be invalid.


Unclear definition of new cluster when limiting egress traffic for AKS

In the purple Note box, it states "Limiting egress traffic only works on new AKS clusters. For existing clusters, perform a cluster upgrade operation using the az aks upgrade command before you limit the egress traffic."
It is unclear in the documentation what defines a "new AKS cluster".  Version?  Date of creation?

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

ID: a38439d0-0125-3445-1c0a-a03b940e315d
Version Independent ID: c6436a35-94b9-6396-3b13-9291f0a17b21
Content: Restrict egress traffic in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/limit-egress-traffic.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@techbunny Thanks for the Feedback.
Similar feedback was received in past on clarifying what a new AKS cluster mean. The Feedback has been shared to the document author for updating the document on this.
However, I am linking this issue to the old issue #46588 where PG has added clarification on this.
Below are PG comments:
"Old" or "existing" cluster means any cluster using authorized IP ranges during preview. The existing rules and behavior will continue to work, but new updates or newly created clusters after the feature went GA have a dependency on SLB. The best path would be to recreate the cluster to use the latest infrastructure such as SLB, today this can be done in a migration pattern but AKS is investigating a way to make this automated/1-click for you.

@mlearned Please update the document to clarify on the new Cluster part.

@techbunny I am proceeding to close this issue. If there are any further questions regarding this, requesting your to add on the linked issue here.
Thanks.


[rest core] add HttpResponse and AsyncHttpResponse

look at httpx
saving context managing and iter_lines() for a later issue

added


Add missing isStale property in api.md

:tada: This PR is included in version 2.5.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Found a YT link what cannot download

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

WARNING
All invalid issues will be rejected!!

Before going further

If your problem is a bug with youtube-dl or a request for new site support please report it here

Make sure you are using the latest youtube-dl-gui version (Click the Settings icon and then About to view the current version)

Make sure you are using the latest youtube-dl version (Click the Settings icon and then Update to update to the latest youtube-dl version)

Make sure you searched the bugtracker for similar issues including closed ones

Make sure to read the FAQs file

[ ] I think my problem is NOT with youtube-dl
[x] I've verified and i assure that I'm running youtube-dl-gui 0.4
[x] I assure that i am using the latest version of youtube-dl
[ ] Searched bugtracker
[ ] I've read the FAQs file

What is the purpose of your issue?

[x] Bug report
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Please remove any sections between (---) if they are not related to your issue

Bug report
If the problem occurs when downloading a URL please provide the full verbose output as follows:

Restart youtube-dl-gui
Go to Options > Extra tab
Enable Debug youtube-dl
Go to Options > Advanced tab and Clear your log content
Try to download the URL
Copy the whole log content and insert it between the ``` part below

delete me and insert your log content here

What operating system do you use ?
List of actions to perform to reproduce the problem:

..
..
..

What is the expected behaviour ?
What happens instead ?

Feature request (request for a new functionality)
Please make sure that the requested feature is NOT already in the TODO list

[ ] I've verified and i assure that my requested feature is NOT in the TODO list

What operating system do you use ?

I found a link what youtube-dlg cannot download . It reports "Error" in dialog box. https://www.youtube.com/watch?v=Ir7pVCwFFj4  Other downloaders can donload it.
Update:
While I am playing it in opera browser, the downloading was completed.
[09/18/18 13:57:00] WARNING: Falling back on generic information extractor.
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\youtube.py", line 1210, in _decrypt_signature
[09/18/18 13:57:00] ERROR: Signature extraction failed: Traceback (most recent call last):
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\youtube.py", line 1121, in _extract_signature_function
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\youtube.py", line 1210, in _decrypt_signature
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\youtube.py", line 1182, in _parse_sig_js
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\youtube.py", line 1121, in _extract_signature_function
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\common.py", line 972, in _search_regex
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\youtube.py", line 1182, in _parse_sig_js
[09/18/18 13:57:00] youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[09/18/18 13:57:00]   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsjdqsmfs\build\youtube_dl\extractor\common.py", line 972, in _search_regex
[09/18/18 13:57:00]  (caused by RegexNotFoundError('Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[09/18/18 13:57:00] youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[09/18/18 13:57:00]  (caused by RegexNotFoundError('Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[09/18/18 14:02:55] WARNING: video doesn't have subtitles
[09/18/18 14:04:34] WARNING: video doesn't have subtitles
[09/18/18 14:04:34] WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[09/18/18 14:52:49] WARNING: video doesn't have subtitles
[09/18/18 14:52:49] WARNING: Requested formats are incompatible for merge and will be merged into mkv.


Chat base dashboard is not showing any message reports (handeld, unhandeled, active) after sending status ok message.

here is the screen shots when sending request and after sending request....for your information i updated the time stamp to refer todays date may be if its an error.

Recently I see the same issue. I see that redirect urls are reported but users don't.

@danitsi Have you found the solution to fix it?


Device not showing up?

I have a MR101Z I installed. I've followed all the proper steps setting  up the device handler, the fan works and the remote works but my Smartthings hub isn't seeing the device. I've setup other devices with custom device handlers, so I don't think there's an issue there. I'm not saying it's a code issue, but is there any more troubleshooting I can try, or anything about my hub that might be the issue? I also brought the hub feet from the fan control antenna, but I have plenty of repeater devices so I don't think that'd be an issue either.
Hub Firmware Version |<PHONE_NUMBER>2
ZigBee | State: Functional
Version: 2.2.22
EUI: 24FD5B0002007D52
Channel: 11
Node ID: 0000
Pan ID: 2EC8
OTA: disabled
Unsecure Rejoin: true
Z-Wave | State: FunctionalHome
ID: C258BBF7
Node ID: 01
Suc ID: 01
Protocol Version: 4.05
Region: US

I am having the same problem.

On Sep 19, 2020, at 5:48 PM, Mat<EMAIL_ADDRESS>wrote:

I have a MR101Z I installed. I've followed all the proper steps setting up the device handler, the fan works and the remote works but my Smartthings hub isn't seeing the device. I've setup other devices with custom device handlers, so I don't think there's an issue there. I'm not saying it's a code issue, but is there any more troubleshooting I can try, or anything about my hub that might be the issue? I also brought the hub feet from the fan control antenna, but I have plenty of repeater devices so I don't think that'd be an issue either.
Hub Firmware Version |<PHONE_NUMBER>2
ZigBee | State: Functional
Version: 2.2.22
EUI: 24FD5B0002007D52
Channel: 11
Node ID: 0000
Pan ID: 2EC8
OTA: disabled
Unsecure Rejoin: true
Z-Wave | State: FunctionalHome
ID: C258BBF7
Node ID: 01
Suc ID: 01
Protocol Version: 4.05
Region: US
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

We are on hub version<PHONE_NUMBER>6. I would say the issue should be
caused by your hub firmware version.
Rafael Borja
<EMAIL_ADDRESS>On Sat, 19 Sep 2020 at 20:14, gfa100856<EMAIL_ADDRESS>wrote:

I am having the same problem.

On Sep 19, 2020, at 5:48 PM, Mat<EMAIL_ADDRESS>wrote:

I have a MR101Z I installed. I've followed all the proper steps setting
up the device handler, the fan works and the remote works but my
Smartthings hub isn't seeing the device. I've setup other devices with
custom device handlers, so I don't think there's an issue there. I'm not
saying it's a code issue, but is there any more troubleshooting I can try,
or anything about my hub that might be the issue? I also brought the hub
feet from the fan control antenna, but I have plenty of repeater devices so
I don't think that'd be an issue either.
Hub Firmware Version |<PHONE_NUMBER>2
ZigBee | State: Functional
Version: 2.2.22
EUI: 24FD5B0002007D52
Channel: 11
Node ID: 0000
Pan ID: 2EC8
OTA: disabled
Unsecure Rejoin: true
Z-Wave | State: FunctionalHome
ID: C258BBF7
Node ID: 01
Suc ID: 01
Protocol Version: 4.05
Region: US
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rafaelborja/SmartthingsKingOfFansZigbee/issues/16#issuecomment-695370032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGVTBBP5YSVQVV7M6CTR7DSGVCPLANCNFSM4RTH6JEA
.


Minion dmg/victarios charity

dmg calculation for Raise zombie slam (w/ violent dead x2) doesn't take into account shared charges from Victario's charity.
build:
https://pastebin.com/xfe3CqU3

The 1.4.3 update added options to the Configuration tab to enable charges for minions.


python311Packages.argcomplete: 3.1.6 -> 3.2.1

Diff: https://github.com/kislyuk/argcomplete/compare/refs/tags/v3.1.6...v3.2.1
Changelog: https://github.com/kislyuk/argcomplete/blob/v3.2.1/Changes.rst
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build python312Packages.pytest python312Packages.jsonargparse python312Packages.pipx python311Packages.censys python311Packages.pytest python311Packages.knack esphome azure-cli cwltool


Description for Objectives

It would be a great addition to have descriptions or journals link to them for individual objectives.

You can paste any document links into objectives including links to journals.
Check this section from the FQL video tutorials:
https://www.youtube.com/watch?v=Dn2iprrcPpY&list=PLHslnNa8QKdD_M29g_Zs0f9zyAUVJ32Ne&index=3&t=459s
If you haven't watched the tutorials there is a lot of good information about the features of FQL. You can examine the video details to get timecode links to topics:
https://www.youtube.com/playlist?list=PLHslnNa8QKdD_M29g_Zs0f9zyAUVJ32Ne

It's a bit quixotic w/ v10 that there is a copy document ID button on the left hand side of document sheets including journals. I wished they included a button to copy the UUID / document link as well. The trick is to open the TinyMCE editor on the quest details page; drag and drop the document you want to link in the objective into the editor then cut / paste that when you start editing an objective.


View and Add Post Comments

Vercel build is failing because this branch was created before moving the _container file to a valid location.


Reset firstId even if app is in the foreground

firstId are only reset if the app is in the background:
https://github.com/snowplow/snowplow-android-tracker/blob/b702375c004dad67da84ffd9961c486e3435aaa0/snowplow-tracker/src/main/java/com/snowplowanalytics/snowplow/tracker/Session.java#L140-L147
which causes subsequent foreground sessions to have the same firstId

c.f. #192

Ouch good catch, thanks for scheduling

and c.f. https://snowplow.zendesk.com/agent/tickets/8211


Why we need to "ADD NEW ADDRESS"again and again?

What I'm trying to achieve
…
Thanks for your huge work,this project is really an artwork.
I don't want to input my address again and again after I have fully charged many orders
could I require such a feature from you geniuses~?
Describe a proposed solution
could you add an button in the picture such as "use previous address"
if it's convenient to you or when you have time?
Much Thanks
Other solutions I've tried and won't work
…
Screenshots or mockups

It's implemented, but you need to log in to a user account to use that feature.
Here you can log in:

After placing an orders as a registered customer your addresses will be saved and you'll be able to reuse them later:

@maarcingebala
Thanks for your replies.
To test it,I create a new account....
I found:
after the new account make an order(status:charged in saleor-front and fufiled  in saleor-dashboard)
then,
the new account must log out,otherwise,previous address will not occur after I press F5 buttion many times,is this my false impression?

@appleyuchi Thanks for reporting. We'll investigate it as soon as we can. Once I have more information regarding this issue, I will update you.

Thank you ~

@appleyuchi We have recently released new checkout, and I have verified it fixed the issue, logout and login is now not required to update available addresses in checkout.


refactor(v2): add exception handling if external command is fails

Motivation
Related to #4844.
In fact, we have exception handler for internal commands, so I think it makes sense to do something similar for external commands.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Not yet possible
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks
Not sure it's the perfect solution yet, but at least it works in both cases so we can merge it for now


Spelling mistake in FAQ section

Describe the bug
Spelling mistake in the FAQ page where "ReactJs is spelt as "RaectJs"
To Reproduce
Steps to reproduce the behaviour:

Go to the faq section
Click on "Can I use anything other than React?"
The error is present in the answer.

Expected behaviour
The spelling must be "ReactJs"
Screenshots

Go ahead and fix it @ashrrithk
Also, do an overall review of the texts so that can fix other mistakes too.

@atapas I went through the texts on the landing page. It's all okay, except for the spelling that I mentioned.

Sure @ashrrithk fix that one then.


Addons w/ initial tiller and dashboard implementation

What this PR does / why we need it: Adds a general "addons" array to KubernetesConfig to standardize the way we configure Kubernetes addon deployments/daemon sets. Each addon entry can define an arbitrary set of containers, each with config key/vals, mimic'ing the general pattern for a Kubernetes deployment or daemon set.
This is how you would enable an addon without providing any customizations (accepting the acs-engine-provided defaults):
"kubernetesConfig": {
        "addons": [
          {
            "name": "tiller",
            "enabled" : true
          }
        ]
      }

This is how you would enable and customize the configuration of an addon (using tiller as an example) in your kubernetesConfig:
"kubernetesConfig": {
        "addons": [
          {
            "name": "tiller",
            "containers": [
              {
                "name": "tiller",
                "image": "myDockerUser/tiller:customSHA",
                "cpuRequests": "100m",
                "memoryRequests": "500Mi",
                "cpuLimits": "100m",
                "memoryLimits": "500Mi"
              }
            ]
          }
        ]
      }

This is how you disable an addon:
"kubernetesConfig": {
        "addons": [
          {
            "name": "tiller",
            "enabled": false
          }
        ]
      }

The above implementation allows the following use-cases:

enable a Kubernetes addon deployment by default, with default configuration
enable user-tunable configuration of default Kubernetes addons
provide a simple switch to turn off a default Kubernetes addon
provide a Kubernetes addon deployment w/ default configuration as an opt-in, but disable by default
provide a simple switch to turn on an optional Kubernetes addon, including optional user-tunable configuration

Special notes for your reviewer: Needs discussion and unit test coverage.
Fixes #1748
Release note:

general Kubernetes addons configuration interface

Seems pretty reasonable. On/off and basic knobs seem appropriate, anything more complicated than that should likely be pushed to helm charts (methinks). 👍

@jchauncey I'd like your code review on this one

Thanks @CecileRobertMichon good catch!


Changing docs and examples to use correct function signature

P: AsRef<[ChildNumber]> means that path can be borrowed as a reference to a slice of ChildNumber.
While it does work with Vec, the documentation and examples mislead the user into thinking that he/she has to use a vec, which is not true. A simple reference to a slice of ChildNumber also works and does not consume heap

Added 0.30 milestone so that the documentation is updated.


Art not staying when set.

What TMM version are you using?
3.0-SNAPSHOT
release, pre-release, nightly, or directly from GitHub/branch?
Build date: 2018-09-20 00:35
What is the actual behaviour?
Not able to set artwork and have it stay after closing program. I changed series poster and fanart as well as season posters and thumbs. I also made sure to rewrite the nfo when done.
What is the expected behaviour?
Have artwork stay after setting it.
Steps to reproduce:

Select local art for poster.
If it takes it, close the program and open it back up.
The artwork is gone.

Additional
tmm.2018-09-19.5.log.gz
tmm-1.mkv.zip
Have you attached the Logfile from the day it happened?
Yes. I also uploaded a video to better explain what is happening.
Your Operating system? (win/mac/linux? version?)
Arch Linux: 4.18.7-arch1-1-ARCH

After some more testing I think it might be related to my art folder being hidden.
.art_bu
I'm able to browse to and pick the art I want without any trouble, and the correct path is listed in the artwork URL, but it just won't load it. If I move the art to a non hidden folder, I can select it without any trouble.

we've migrated v3 to GitLab (https://gitlab.com/tinyMediaManager/tinyMediaManager) along with all issues. The issues for v2 at GitHub will be closed - we will respond to the corresponding issue in GitLab in the future
br
Manuel


Identifier set, but id sent

Hi there,
I started to use ng-admin few days later to quick scaffold a nice GUI admin with an API built with Hapi and Mongoose. But I encountered an issue with the Entity.identifier() method.
I have set my identifier field as following :
var user = new Entity('user').identifier(new Field('_id'));

But here is the request from the edit form :
Request URL: http://localhost:3000/v1/user/5482e8405e6d4bbf1d8656f1
Request Payload:
    email<EMAIL_ADDRESS>    id: "5482e8405e6d4bbf1d8656f1"
    language: "548342cd7d4ada54720d9cc0"
    password: "password"
    pseudo: "UserTest"

As you can see the identifier is correctly set on the URL, but the payload still have the default one as id instead of _id.

So what was the fix? Is there an additional option?


348 Fetch site statuses from Find API

Context
We want to stop candidates from applying to courses that are full, so that they don't waste time on their application unnecessarily. The first step in doing so is to retrieve data from the Find API which tells us the vacancy status of a given course/site choice.
Changes proposed in this pull request
Update the SyncProviderFromFind rake task to pull in and persist the required data. See commits for more detail.
Guidance to review

A per-commit review will step through the precursory refactors before finally arriving at the described change.
Testing locally, run bundle exec rake sync_dev_providers_and_open_courses. Inspecting your database, records in course_options should be populated with a vacancy_status value that matches the enum decleration in the CourseOption model.

Link to Trello card
https://trello.com/c/LAh0iGIc
Things to check

[ ] This code doesn't rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

Missed a failure in spec/system/support_interface/providers_spec.rb, fixing now.


Android API for instant click handlers?

Had a discussion with @sargunv on Slack.
It's pretty hard to register a click handler you want to register immediately (without disambiguating double clicks).
Should this be we add some new API for this?
https://github.com/user-attachments/assets/39066492-417f-49b9-9809-940e246275e4
Here I copy pasted the implementation of onSingleTap to  onSingleTapConfirmed to and commented out the body of onSingleTap.

Example of how to do this without a new API
// map is a MapLibreMap
val gestureManager = map.gesturesManager
val tapListener =
  object : SimpleStandardOnGestureListener() {
    override fun onSingleTapUp(event: MotionEvent): Boolean {
      println(event)
      return true
    }
  }
val tapDetector = GestureDetector(context, tapListener)
val tapGesture =
  object : BaseGesture<StandardOnGestureListener>(context, gestureManager) {
    override fun analyzeEvent(event: MotionEvent) = tapDetector.onTouchEvent(event)
  }
gestureManager.detectors.add(tapGesture)

Thanks for sharing. I'll make sure this is added to the documentation.


Link to original project on GitHub

because it has moved from GoogleCode to GitHub.

Thanks for doing that.

Thanks


Visual Studio Code "Could not copy "obj\Debug\netcoreapp3.1\WebAbApi.dll" to "bin\Debug\netc Api.dll" to "bin\Debug\netcor during build

C:\Program Files\dotnet\sdk\6.0.404\Microsoft.Common.CurrentVersion.targets(4632,5): error MSB3027: Could not copy "obj\Debug\netcoreapp3.1\SIS_WebApi.dll" to "bin\Debug\netcoreapp3.1\SIS_WebApi.dll". Exceeded retry count of 10. Failed. The file is locked by: ".NET Host (7196)" [C:\dotnet\...................\SIS_WebApi\SIS_WebApi.csproj]
C:\Program Files\dotnet\sdk\6.0.404\Microsoft.Common.CurrentVersion.targets(4632,5): error MSB3021: Unable to copy file "obj\Debug\netcoreapp3.1
\SIS_WebApi.dll" to "bin\Debug\netcoreapp3.1\SIS_WebApi.dll". The process cannot access the file 'C:\................\bin\Debug\netcoreapp3.1\SIS_WebApi.dll'
 because it is being used by another process. [C:\...................................]
    10 Warning(s)
    2 Error(s)

I have been trying to run.
dotnet build

dotnet run

As there is no Errors showed by Omnisharp.
But still not compiling.

I have encountered similar error messages in Visual Studio 2013.
Mostly, I have found that this situation has occurred when a debug process was halted because of an exception.
When clean+build has not resolved this problem for me, I have had success by doing the following:

Closing Visual Studio
Deleting the bin and obj folders, and
Reopening Visual Studio.

This "bug" has existed since Visual Studio 2003.
Finally, I have also found that I can often overcome this problem by simply renaming the executable file and then deleting it.


Add lower bound for groupCubeSize

When the number of elements in a table is large, the groupCubeSize used by CubeWeightsBuilder tends to 1. This can leads to a large number of estimated Cube Weight pairs that can cause OOM errors.
For instance, using default values for bufferCapacity and desiredCubeSize, a table with 2600M records split between 1200 partitions, the groupCubeSize is 190. The algorithm estimates 37K cubes, among which only around 1K are used. The workaround is to set a minimum value for groupCubeSize, for instance 1000, so the number of estimated cubes won't be as large. Using this setting, the number of estimated cubes is reduced to about 5K.
Test added:
Using default bufferCapacity and DesiredCubeSize, make sure that the minimum value for groupCubeSize is respected for different numElements.

The sampling test failed because now the groupCubeSize is 1000 and there are only a handful of cubes. Use a sampling fraction of 0.01 can solve the problem, should be modify the test? @osopardo1

This test in particular only checks if the filtering of the files is done correctly. Yes, is best to do it with a sample of 0.01.
But then we need to test better the number we are using to solve the issue. Or at least, test it in corner cases.


Readme mentions we can set TransportationMode in the builder, but it is not available

Readme for the Publisher SDK Usage says it’s possible to set a TransportationMode, though it’s not available in the api, and IDE shows an error: Cannot find 'TransportationMode' in scope. I see that instead we can use  .routingProfile(RoutingProfile.cycling).  I'm talking about this bit:
publisher = try? PublisherFactory.publishers() // get a Publisher Builder
    .connection(ConnectionConfiguration(apiKey: ABLY_API_KEY, clientId: CLIENT_ID)) // provide Ably configuration with credentials
    .log(LogConfiguration()) // provide logging configuration
    .transportationMode(TransportationMode()) // provide mode of transportation for better location enhancements
    .delegate(self) // provide delegate to handle location updates locally if needed
    .start()

The corresponding Jira issue is https://ably.atlassian.net/browse/SDK-1953


Timezones support

bah, humbug!

So we should try to detect the browser timezone and shift displayed datetimes to match. Will have to add moment.timezone to the npm package and require it into the js app

Already set up to display local time
From machine on PST:

From machine on EST:


Unused dependency

I use this boilerplate quite a lot and I like it, I've been looking through package.json and it seems to me this dependency is unused:
"babel-preset-react-hmre": "^1.1.1"

I'll remove this soon


bump version number to 0.7.0

Did you guys release a new version by mistake?
https://rubygems.org/gems/hubspot-ruby lists the version 0.7.0 now

If this was not by mistake, this will fix the version number in the hubspot-ruby.gemspec file.

We actually released 0.7.0 from the v0-stable branch since there are a bunch of changes on master that aren't ready to be released yet. That's why you don't see the version number in the gemspec updated on master but you will on v0-stable. I suppose it doesn't hurt to also have the version number bumped on master.


Add clones and downloads badges to README.md

@mosheraboh After I merged this in, the PyPi downloads badge was not working correctly so I rolled back the PR. Could you submit another PR with a working version and then I can merge it in. The GitHub Clones badge seems to be working right.


Added normalizePath

Added normalizePath method from vite so the import part for the templates work on windows machines. Changes e.g. c:\path\to\file.twig => c:/path/to/file.twig
Issue: https://github.com/larowlan/vite-plugin-twig-drupal/issues/24

@larowlan Just a wee PR to add the normalizePath method to the resolveFile method.
Sorry, I don't know how to properly link the PR to the ticket, so I just added the link the PR comment.

Made a minor change, as I prefer early returns. If it passes tests happy to ship this


Import from S3 without Access key but with IAM role

For Exasol instances hosted in AWS, you could specify an IAM role to grant them read/write access to S3. Therefore, it would be good to have an option to import without Access key and secret but with making use of that role.

As discussed internally, we are not going to implement this feature.


Fix custom_target_dependency test.

The feature was renamed in https://github.com/rust-lang/rust/pull/79336

r? @Eh2406
(rust-highfive has picked a reviewer for you, use r? to override)

CI will fail until https://github.com/rust-lang/rust/issues/79459 is fixed.

Also, I'm uncertain why the Freeze trait is being defined at all.  It is not needed to compile, but it looks like it was included with a purpose, maybe to verify that you can redefine a lang item?

@bors: r+
I think at one point in the past it was required to be defined, but it seems like not anymore?

:pushpin: Commit 8d230f958aab1fd4e2781f96f146c48adc998aee has been approved by alexcrichton

@bors: r+

:bulb: This pull request was already approved, no need to approve it again.

:pushpin: Commit 8d230f958aab1fd4e2781f96f146c48adc998aee has been approved by alexcrichton

:hourglass: Testing commit 8d230f958aab1fd4e2781f96f146c48adc998aee with merge 502ced2983277c07d13139968930dd49f7eb44af...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 502ced2983277c07d13139968930dd49f7eb44af to master...


[FFM-1549] Evaluation engine and integration of ff test cases

Introduce new package pkg/evaluation
Old evaluation still exists
Remove evaluation types and use single source of truth (rest types)
Introduce repository
Store Interface added, TBD local storage implementation

fyi @jcox250 after this the sdk will be saving the rest types directly to the cache, so we won't need a lot of the wrapper code or type conversion code in the proxy anymore which will be great

@enver-bisevac looks like some of the basic checks are failing on this one


useSession on client return cached session instead of fetching

Is this suited for github?

[x] Yes, this is suited for github

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
If I'm using the useSession hook, whenever I refresh the page it sends an API request to get-session. We should already have the session locally, though. A comparison would be Supabase, their useSession hook does not send any requests unless the session is expired. Having to fetch the session when it already exists locally will make sites feel slower on initial load, and incur unnecessary bandwidth on the API. useSession needs to return the cached session on mount
Describe alternatives you've considered
useSession hook return cached session immediately on client, only fetch get-session if it is expired.
Additional context
No response

The session cache is stored as an HTTP-only cookie, and the server needs to validate that the token is properly signed before sending back the session data. If there is no API request, it would mean storing the session in a way that js can read it, which isn’t ideal and is not recommended. Supabase do this mostly cause they don't have any other option since 3rd party cookies nowdays get blocked by browsers.

Makes sense. I'm also seeing that it's unsafe to store JWT in localStorage so I need to update some things there. Is it possible to have get-session return the jwt as well? I also saw examples of storing Bearer in localStorage, isn't that vulnerable to XSS attacks?

Yeah, it’s vulnerable to XSS, but it’s all about trade-offs. If an attacker manages to pull off an XSS attack, they can already send requests directly to your server, so stealing tokens might not even be even that important. They can do a lot of things. So, locking down your site against XSS should be the first priorit and it’s pretty straightforward these days.  And, JWTs can easily get too large for cookies, so local storage makes more sense. It’s not perfect, but as long as you’re aware of the risks and have proper security measures in place, it’s a workable option.

In this case wouldn't it also be fine to store the session cached so it can be used immediately on page load, so other requests (e.g. getMyProfile /api/profile/me) aren't dependent on waiting for that first API request to resolve. Something like useOptimisticSession that gets revalidated instead of starting as null

The only concern I think with using localStorage is storing PII like email? But storing the JWT would also do that in its current state?

I think this should currently be handled by users rather than from the lib. Specifically, revalidating the cache becomes a problem. for example, if the user updates their info, the cookie cache will get overridden by the server. To do the same on the client, we would then need to track which endpoint is being calle and invalidate it which introduces a lot of complexity.

Since I'm just caching sessionData and refreshing it wouldn't it be the same outcome though? It still calls getSession on page load, which gets the latest session data and overrides the localStorage cache with the new data

Yea I know I can write the hooks, but I was kind of hoping to see this built in since everyone would benefit from the useSession hook having these features, both devs and all users of any site that has better-auth. Optimized out of the box without needing custom hooks on top of it. If there's no chance of this getting added to the library itself then I may just make a better-auth-swr package that can be used alongside it, but I feel like anyone using better-auth on a React based project definitely needs these features

Oh yeah, the library idea would be great. And if we had a dedicated React package, this would definitely make sense. The only thing we do for each framework (Vue, React, Svelte...) is provide a very lightweight adapter. For example, for the client, we just call the nanostore atoms with the respective useStore hooks and nothing else. In the future, we may have a dedicated adapter, but for now, since we're keeping better-auth as a single package, we try to avoid framework-specific code as much as possible.

Better-auth agnostic, it's great. I think also this should currently be handled by users (or another lib) rather than from better-auth lib.
I use Vue.js and React, the two have a different logic. If it can be useful to someone, here's how I do it with Vue.js:
watch(() => currentSession.value, (newVal) => {
    if (newVal.session.isPending || newVal.session.isRefetching) {
      appStore.user.value = null
      return
    }

    appStore.user.value = newVal?.user
  }, { immediate: true })

I don't think Vue uses the useSession React hook, the feature suggestion was for the React hook itself to reduce amount of "auth code" in the application

UseSession() is a Nanostore, not a React hook. So Vue.js use also useSession() provided by Better-auth.
A different package for React would be relevant, given that React isn't everything, there's also Vue.js, Solid.js, Angular, htmx, Alpine, Astro, etc. So a list of external adapters is more clean, instead of bloat the main package with several useless dependencies.

In this case isn't localStorage agnostic, and can be used within useSession as an option to benefit every framework

yeah that would be cool for offline First (pwa/capacitor) applications
also i found solution https://github.com/onejs/one/tree/af7e4049000f60e41a7b231d56f16d0393f72abc/packages/better-auth
but only for React


updated punctuations

Punctuations are corrected in the readme file.

We will not be encouraging one liner grammatical changes as this is expensive process, thank you for your interest.
CC @mihaimaruseac


Proposal for Issue on Issue-719

Proposal fix for issue 719
cc @robshakir @xw-g

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.588%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1602

Relevant Lines:
2177

💛  - Coveralls

cc: @jjing1122

During local run of route_propagation_test, i hit the following errors, it seems oc path is invalid. have you seen this error or the test passed in your local test environment?
"error receiving gNMI response: failed to render notifications: cannot represent field value [0xc0049bff60] as TypedValue for path /network-instances/network-instance[name=port2]/protocols/protocol[identifier=BGP][name=0]/bgp/rib/attr-sets/attr-set[index=2]/as-path/as-segment: invalid type ptr in leaflist"
based on the codes, it seems oc path should be something like :
/network-instances/network-instance[name=port2]/protocols/protocol[identifier=BGP][name=0]/bgp/rib/afi-safis/afi-safi[afi-safi-name=IPV4_UNICAST]/ipv4-unicast/neighbors/neighbor[neighbor-address:xxxx]/adj-rib-in-pre/routes/route[path-id=0][prefix=xxxx]/state/attr-index

The error you mentioned is from ixgnmi.go:
E1115 12:10:51.229043   22905 ixgnmi.go:469]
Failed to publish roots:
failed to render notifications: cannot represent field value [0xc0026fefc0] as TypedValue for path:
/network-instances/network-instance[name=port2]/protocols/protocol[identifier=BGP][name=0]/bgp/rib/attr-sets/attr-set[index=0]/as-path/as-segment:
invalid type ptr in leaflist

which is generated by this block (just pasting prefix.v4 for the sake of simplicity):
https://github.com/openconfig/featureprofiles/blob/45481794a66a41bb6dfea1bec98cdd4cd84d5dfc/feature/bgp/addpath/ate_tests/route_propagation_test/route_propagation_test.go#L310-L321
We only see this issue when testing our changes with yesterday's commit https://github.com/openconfig/featureprofiles/commit/d3d4ff7252906141715820ad0c48492f828c9a87. However, that gNMI path doesn't seem like it's returned from the DUT but returned by the ATE.
When this was tested a week ago, everything worked as expected.

@jjing1122  The error mentioned is not related to this PR.  Any other concerns about this PR? LGTM?
@vcheny @sachendras @vosipchu, BGP-PEER-GROUP1 is referenced but not defined. This PR is to fix that. I think probably we only need 1 LGTM from one of you, just in case I missed anything before I merge it.

@Hobbydos can you also help update the OTG part? Thanks.

Xiao, i see both groups, definitions and policies applied. LGTM

LGTM.

/gcbrun

@jjing1122 conflicts should be handled.


My Services accordion is occasionally empty

Sometimes when you go to my-services page the accordion will be empty (besides title) and there will be no published/unpublished tabs. Will add screenshot if I can reproduce.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7
┆Sprint: Seabright Sprint 18 Skipjack
┆Issue Number: DOCK-891

Is it possible the accordion was just collapsed?

@garyluu No it was not.

Managed to reproduce this a total of one time.  Some observations:
The accordion is not expanded. Trying to expand it, causes it to immediately collapse. Other panels seem to workflow fine.

Lowering priority to 1.7.x, if this is so hard to reproduce I'm ok with it being a potential hotfix

To kind of consistently reproduce (on my machine) while developing.  Open 10 tabs to /my-tools .  Change anything in the code to trigger a refresh on all tabs (because of watch).

Might be fixed? Tried a bunch to reproduce and seems okay...maybe. Closing this and hoping it doesn't come back.
API - 1.8.0-alpha.2
UI - 2.5.0-alpha.0


Issue 229

This should be ready for merge, unless there are more issues. It will not affect anything until the css packages are updated.

Hei Silvi, I'm sorry this remained lost among other issues...
I'll try to have a look at this tomorrow.

Actually hold up on that. Sure look at it but I have been thinking and maybe just the link would work. I happened to be logging into the iOS Tumblr app and their log in is absolutely beautiful. Before adding this we may want to revisit how devs configure the buttons and links.

:+1:
Ok, lets try to break down things and design a new system for v2.0!

We'll reopen this in case we'll wish to proceed further!


Add a cookbook section to the README

This is amazing, thanks for organizing the cookbook @ericfischer.  cc/ @samgehret @allanwalkerit @cmtoomey.


after the palera1n screensaver appears, the regular os loads and also writes that it is blocked

Description
I've already tried a lot of things, but everything leads to the same result

what do you mean by "writes that it is blocked", can you show what you mean?


Tomcat security permissions

At the moment a user of vcap can access/read/write files and directories outside the the web context.

Hi Justin,
Please use our Cloud Foundry community site to post questions and issues regarding Cloud Foundry: http://support.cloudfoundry.com/
The specific issue you're seeing may be resolved by configuring the Cloud Controller and DEAs to run in secure mode: http://support.cloudfoundry.com/entries/20067403-how-to-activate-secure-mode-on-cloud-foundry


Service discovery errors during startup in v2.13.0

Bug Report
What did you do?
We've upgraded Prometheus from 2.6.1 to 2.13.0.
When Prometheus is starting up there are service discovery errors logged, partly because of context cancellation. Service discovery otherwise works.
Also it seems the config is read twice.
What did you expect to see?
No errors during startup.
What did you see instead? Under which circumstances?
level=info ts=2019-10-08T15:01:57.136Z caller=main.go:743 msg="Loading configuration file" filename=/etc/prometheus/config/prometheus.yml
level=error ts=2019-10-08T15:01:57.138Z caller=pod.go:85 component="discovery manager scrape" discovery=k8s role=pod msg="pod informer unable to sync cache"
level=error ts=2019-10-08T15:01:57.138Z caller=refresh.go:78 component="discovery manager scrape" discovery=ec2 msg="Unable to refresh target groups" err="could not describe instances: RequestCanceled: request context canceled\ncaused by: context canceled"
level=error ts=2019-10-08T15:01:57.138Z caller=node.go:82 component="discovery manager scrape" discovery=k8s role=node msg="node informer unable to sync cache"
level=error ts=2019-10-08T15:01:57.138Z caller=refresh.go:78 component="discovery manager scrape" discovery=ec2 msg="Unable to refresh target groups" err="could not describe instances: RequestCanceled: request context canceled\ncaused by: context canceled"
level=info ts=2019-10-08T15:01:57.138Z caller=kubernetes.go:192 component="discovery manager scrape" discovery=k8s msg="Using pod service account via in-cluster config"
level=info ts=2019-10-08T15:01:57.139Z caller=kubernetes.go:192 component="discovery manager scrape" discovery=k8s msg="Using pod service account via in-cluster config"
level=info ts=2019-10-08T15:01:57.178Z caller=main.go:771 msg="Completed loading of configuration file" filename=/etc/prometheus/config/prometheus.yml
level=info ts=2019-10-08T15:01:57.182Z caller=main.go:743 msg="Loading configuration file" filename=/etc/prometheus/config/prometheus.yml
level=error ts=2019-10-08T15:01:57.183Z caller=pod.go:85 component="discovery manager scrape" discovery=k8s role=pod msg="pod informer unable to sync cache"
level=error ts=2019-10-08T15:01:57.183Z caller=node.go:82 component="discovery manager scrape" discovery=k8s role=node msg="node informer unable to sync cache"
level=error ts=2019-10-08T15:01:57.183Z caller=refresh.go:78 component="discovery manager scrape" discovery=ec2 msg="Unable to refresh target groups" err="could not describe instances: RequestCanceled: request context canceled\ncaused by: context canceled"
level=info ts=2019-10-08T15:01:57.183Z caller=kubernetes.go:192 component="discovery manager scrape" discovery=k8s msg="Using pod service account via in-cluster config"
level=error ts=2019-10-08T15:01:57.183Z caller=refresh.go:78 component="discovery manager scrape" discovery=ec2 msg="Unable to refresh target groups" err="could not describe instances: RequestCanceled: request context canceled\ncaused by: context canceled"
level=info ts=2019-10-08T15:01:57.184Z caller=kubernetes.go:192 component="discovery manager scrape" discovery=k8s msg="Using pod service account via in-cluster config"
level=info ts=2019-10-08T15:01:57.224Z caller=main.go:771 msg="Completed loading of configuration file" filename=/etc/prometheus/config/prometheus.yml

I checked prometheus_sd_refresh_duration_seconds and it doesn't show any abnormal values at startup.
prometheus_sd_refresh_failures_total stays at 4 after startup.
The nodes were doing a fairly long WAL replay before this.
Environment

System information:
We are running the equivalent of the upstream Docker image, but changing the user.

Prometheus version:
prometheus, version 2.13.0 (branch: HEAD, revision: 6ea4252299f542669aca11860abc2192bdc7bede)
  build user:       root@188fafb5b41b
  build date:       20191008-10:23:04
  go version:       go1.13.1

Right, I had someone else reporting the same issue just today! I'll send a PR.


Make it possible to resize the diff modal

Hi there,
Great work on monokle so far. I was using the diff option earlier and it was very useful.
Is it possible to make the modal resizable?
It would be useful to see more of the configs if the display size allows for it
When resizing the modal, it should store the size of the modal in % respective to the overall window size. This is so if you have monokle in full screen, resize the modal to 90% width of the window, then resize monokle to be smaller, the modal won't be bigger than the monokle window itself.
The maximum width of the modal should be 90% or 95% such that there is some space on the side.

Thanks!

hey @ioandev that is a really good idea. We will certainly look for it.


Fixes “anchor-is-valid” false positive for hrefs starting with the word “javascript”

Fixes “anchor-is-valid” false positive for hrefs starting with the word “javascript”
Closes #579
Signed-off-by: Anthony Veaudry<EMAIL_ADDRESS>

Coverage remained the same at 99.554% when pulling 830a6761a7517f20f9cae0a8e06b3d1e5f4cf716 on anthony0030:master into 80b070285c2b5f1cebe0752cd9037f9f4f2a3dd9 on evcohen:master.

Is there any ETA for this pull request?


getKeys, when specifying sharedBy, results in creation of a new RemoteSecondary, used only once

Describe the bug
getKeys, when specifying sharedBy, results in LocalSecondary._scan() making a new one-time-use-only connection to the cloud secondary, by creating a new RemoteSecondary object
Steps to reproduce
It's what happens now
Expected behavior
Should use the existing remote secondary
Screenshots
No response
Smartphones
No response
Were you using an atApplication when the bug was found?
No response
Additional context
See also #744

Marked as 5SP in this sprint to include (1) the analysis in #744 (2) the effort in #758


Snapshot, then bootstrap from client-side Vue-generated html?

Hi @stowball! Great project, was just looking for something like this :)
Here's a workflow I am trying to support. I like to build single pages using Vue in its entirety, as I find the component hierarchy and everything else very easy to use, and would like to layer in some interactivity (think, for example, an article that is mostly text and images but has some interaction around charts and data built in).
But for production, I want the rendered page to bootstrap from the Vue-rendered html for SEO and time-to-load purposes, and I don't want to set up true server-side rendering for a single page. But from playing with quench-vue seems to require manual coding of specially constructed html to work, and it feels somewhat tedious to have to use the html as the source of truth for data, templates, and components, or split between the two.
I was wondering if you've thought about some kind of workflow where you do everything in .vue files (specifically, a project created using vue-cli), and then:

Snapshot (within the browser, or using something like phantom-js), the rendered Vue app. Might make sense to do this as a build step.
Output that as the index.html for the production vue app. This should mean that loading index.html even without js enabled would result in the same rendered DOM but without any interactivity.
When instantiating the vue app itself, it would either replace the contents of <div id="app"> (like happens by default) or (ideally) reuse the existing DOM that exists after the rendering of the static content. (not sure of this last one since that

Does this idea make sense? Is this something quench-vue aims to support? Or is that somewhat of an orthogonal project in your mind? I think it's like a mix between https://github.com/bahmutov/hydrate-app and this. Very interested in this concept, would be happy to contribute. cc @bahmutov
Also FYI the project logo on the README is a 404 now it seems :(
Cheers,
f

After some more digging I found https://www.npmjs.com/package/prerender-spa-plugin which does this as a webpack plugin. Carry on :)


Error occurred when I run "make docs"

GEN    distclean-asciidoc-guide
gmake[1]: Entering directory '/Users/linrl3/Desktop/cowboy/deps/asciideck'
 ERLC   asciideck.erl asciideck_attributes_parser.erl asciideck_attributes_pass.erl asciideck_block_parser.erl asciideck_inline_pass.erl asciideck_line_reader.erl asciideck_lists_pass.erl asciideck_reader.erl asciideck_source_highlight.erl asciideck_stdin_reader.erl asciideck_tables_pass.erl asciideck_to_html.erl asciideck_to_manpage.erl asciideck_transform_pass.erl
compile: warnings being treated as errors
src/asciideck_to_html.erl:81: erlang:get_stacktrace/0: deprecated; use the new try/catch syntax for retrieving the stack backtrace
src/asciideck_to_html.erl:123: erlang:get_stacktrace/0: deprecated; use the new try/catch syntax for retrieving the stack backtrace
gmake[2]: *** [/Users/linrl3/Desktop/cowboy/erlang.mk:5232: ebin/asciideck.app] Error 1
gmake[1]: *** [/Users/linrl3/Desktop/cowboy/erlang.mk:4989: app] Error 2
gmake[1]: Leaving directory '/Users/linrl3/Desktop/cowboy/deps/asciideck'
gmake: *** [erlang.mk:5279: doc-deps] Error 2

This is how it goes. It seems that I failed to make asciideck. Does anyone know how to fix it?

Let me patch it quickly.

Done, you'll need to rm -rf deps/asciideck. Note that at this time it still needs the asciidoc/docbook toolchain for building the guide, I haven't finished that part.
It also shouldn't have failed because of warning, that particular bug was fixed a while ago and Cowboy master doesn't fail anymore.

Do we need Asciideck in any other part of Cowboy? I think I could use the online docs instead of making it offline.


Allow skipping the ContactDetails step if users are logged in, with corr...

...ect info already provided.

:+1: thanks


CI fix test

This reverts commit 529d5e88813d984f931d164b39a7cd693ceaecc5.
CHANGE DESCRIPTION
Problem:
EVEREST-0
Short explanation of the problem.
Related pull requests

[link]

Cause:
Short explanation of the root cause of the issue if applicable.
Solution:
Short explanation of the solution we are providing with this PR.
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?

Tests

[ ] Is an Integration test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Codecov Report

Merging #182 (9b06d28) into main (529d5e8) will increase coverage by 0.03%.
Report is 1 commits behind head on main.
The diff coverage is 50.00%.

@@           Coverage Diff            @@
##            main    #182      +/-   ##
========================================
+ Coverage   5.01%   5.05%   +0.03%
========================================
  Files         13      13
  Lines       3009    2990      -19
========================================
  Hits         151     151
+ Misses      2831    2812      -19
  Partials      27      27

Files
Coverage Δ

pkg/version/version.go
31.81% <100.00%> (ø)

pkg/kubernetes/client/client.go
3.35% <0.00%> (+0.07%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add support for alternate form of CopySource

Follow up from https://github.com/boto/botocore/pull/757, this adds support for a dict value for CopySource.  This ensures we automatically handle all cases correctly for the customer because they're explicitly telling us the bucket, key, and optional version id.
s3.copy_object(CopySource={'Bucket': 'sourcebucket', 'Key': 'sourcekey'})

I've also handed a handler to update the documentation for this new format.
I'll add a handwritten example to boto3 shortly as well.
NOTE: I've branched this off of #757 because I had to modify the existing handler for the string value of CopySource so you'll see the diff of both in this PR.
cc @kyleknap @mtdowling @rayluo @JordonPhillips

Looks fine. I had a couple questions on the logic that I would like answered first before we merge it.

@kyleknap Feedback incorporated.  Ready for another look.

Thanks. Looks much cleaner. :ship:


5200 Allow processing of reports after a schema change

This PR fixes a bug that occurs during deployment when a schema is changed in the middle of a report being processed.  See #5200 for details. The result is that:

The processing fails if fields were added to the schema
The wrong data is assigned to a field if a field is removed from the schema (I do not think this has happened yet as we normally add fields)

Test Steps:
I think the unit tests cover this, but here are steps to test locally:

Submit a report and stop the processing of the report, so the queue message is not pulled.  This may be easier to do by adding code in the ProcessFunction to just return a fail status then removing the code later.
Stop ReportStream.
Change the covid-19 schema to add a field somewhere in the middle of the schema.
Restart ReportStream using the new schema.
Verify that no exceptions are thrown and that the report was processed.

Changes

Internal CSV files are read based on the header values of the data

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[x] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[x] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Fixes

Closes #5200

To Be Done
Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Time to review
Total comments

carlosfelix2
17
3h 33m
11

cwinters-usds
17
1d 5h 33m
9

jimduff-usds
12
2h 30m
10

MauriceReeves-usds
12
39m
6

TomNUSDS
12
55m
44

jbiskie
10
1h 27m
0

kevinhaube
9
24m
5

clediggins-usds
9
27m
0

TurkeyFried
8
2h 5m
3

brick-green-agile6
8
10h 4m
3

ahay-agile6
7
22m
0

sethdarragile6
6
20m
1

snesm
5
3h 48m
3

jh765
5
1h 52m
9

JosiahSiegel
4
18m
0

bgantick
3
1h 9m
3

oslynn
3
20m
0

RickHawesUSDS
3
3d 8h 47m
2

jorg3lopez
3
27m
0

rhood23699
3
15m
0

jeremy-page
2
3m
0

whytheplatypus
1
13m
2

acoushawk
1
4m
0

Clearly, we'll have to do a whole bunch of changes to covid-19.schema in the next deployment, just to make sure this works  ;)


Bunch of various errors

Integration Version: Midea Smart AC 2024.5.3
Max connection time set to 90 sec
Static IP Addresses assigned
Hardware:
AC Units: 2x MDV Aroma (Midea Group)
Wifi Modules: NetHome Plus EU-OSK103
Access Points: TP-Link DECO X20 (no energy saving features enabled)
Periodically re-appearing error logs:
2024-05-26 05:06:55.201 ERROR (MainThread) [msmart.base_device] Network error <IP_ADDRESS>:6444: Transport is closing or closed.
2024-05-26 05:06:57.008 ERROR (MainThread) [msmart.base_device] Network error <IP_ADDRESS>:6444: Transport is closing or closed.
2024-05-26 06:16:53.824 ERROR (MainThread) [custom_components.midea_ac.coordinator] Unexpected error fetching midea_ac data
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 315, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/midea_ac/coordinator.py", line 38, in _async_update_data
await self._device.refresh()
File "/usr/local/lib/python3.12/site-packages/msmart/device/AC/device.py", line 324, in refresh
for response in await self._send_command_get_responses(cmd):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/msmart/device/AC/device.py", line 239, in _send_command_get_responses
responses = await super()._send_command(command)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/msmart/base_device.py", line 38, in _send_command
responses = await self._lan.send(data)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/msmart/lan.py", line 585, in send
await self.authenticate()
File "/usr/local/lib/python3.12/site-packages/msmart/lan.py", line 535, in authenticate
assert self._protocol.authenticated
AssertionError

Error Logs when updating max connection time config param:
2024-05-26 21:55:29.768 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry MDV AC Gabinet for select
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 575, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/select/__init__.py", line 106, in async_setup_entry
return await component.async_setup_entry(entry)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 185, in async_setup_entry
raise ValueError(
ValueError: Config entry MDV AC Gabinet (289e08de99e4fb13136a2ea2cd893ee2) for midea_ac.select has already been setup!

 Error: [Errno 104] Connection reset by peer
2024-05-27 02:20:35.130 WARNING (MainThread) [msmart.base_device] Network timeout <IP_ADDRESS>:6444: Connect timeout.
2024-05-27 02:20:35.131 WARNING (MainThread) [msmart.base_device] No response from <IP_ADDRESS>:6444 in 5.000000 seconds.
2024-05-27 03:42:35.199 WARNING (MainThread) [msmart.base_device] Network timeout <IP_ADDRESS>:6444: Connect timeout.
2024-05-27 03:42:35.199 WARNING (MainThread) [msmart.base_device] No response from <IP_ADDRESS>:6444 in 5.000000 seconds.
2024-05-27 03:44:35.200 WARNING (MainThread) [msmart.base_device] Network timeout <IP_ADDRESS>:6444: Connect timeout.
2024-05-27 03:44:35.201 WARNING (MainThread) [msmart.base_device] No response from <IP_ADDRESS>:6444 in 5.000000 seconds.

Local LAN is very stable, no other device has this kind of problems.

Have the same problem:
No response from <IP_ADDRESS>:6444 in 6.000000 seconds. Network timeout <IP_ADDRESS>:6444: No response from host. No response from <IP_ADDRESS>:6444 in 6.000000 seconds. Network timeout <IP_ADDRESS>:6444: No response from host. No response from <IP_ADDRESS>:6444 in 6.000000 seconds.
and
Unexpected error fetching midea_ac data Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 315, in _async_refresh     self.data = await self._async_update_data()                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/midea_ac/coordinator.py", line 38, in _async_update_data     await self._device.refresh()   File "/usr/local/lib/python3.12/site-packages/msmart/device/AC/device.py", line 324, in refresh     for response in await self._send_command_get_responses(cmd):                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.12/site-packages/msmart/device/AC/device.py", line 239, in _send_command_get_responses     responses = await super()._send_command(command)                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.12/site-packages/msmart/base_device.py", line 38, in _send_command     responses = await self._lan.send(data)                 ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.12/site-packages/msmart/lan.py", line 585, in send     await self.authenticate()   File "/usr/local/lib/python3.12/site-packages/msmart/lan.py", line 535, in authenticate     assert self._protocol.authenticated AssertionError

I'm surprised to see the authentication assertion. Can you enable debug logging and post the full logs here when the error occurs again?
Are you blocking the device from internet access at all?

This is a duplicate of #140

Error Logs when updating max connection time config param:
2024-05-26 21:55:29.768 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry MDV AC Gabinet for select
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 575, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/select/__init__.py", line 106, in async_setup_entry
return await component.async_setup_entry(entry)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 185, in async_setup_entry
raise ValueError(
ValueError: Config entry MDV AC Gabinet (289e08de99e4fb13136a2ea2cd893ee2) for midea_ac.select has already been setup!

Thanks. This log doesn't contains any of the more serious errors your previous report had. Can you capture logs until you get the assertion failures?
e.g.
File "/usr/local/lib/python3.12/site-packages/msmart/lan.py", line 535, in authenticate
assert self._protocol.authenticated
AssertionError

Looks like there's is an issue with setting the max connection lifetime. It doesn't take effect until the 2nd connection is created.
Any update on this question?

Are you blocking the device from internet access at all?

I'm experiencing the exact same thing. Getting 20+ times the same error as described here. Which translates to the device not being available in Home Assistant from a couple of seconds to minutes.
This error originated from a custom integration.

Logger: msmart.base_device
Source: custom_components/midea_ac/coordinator.py:38
integration: Midea Smart AC (documentation, issues)
First occurred: 8:06:40 PM (20 occurrences)
Last logged: 8:28:25 PM

Network error <IP_ADDRESS>:6444: Transport is closing or closed.
Network timeout <IP_ADDRESS>:6444: Connect timeout.
No response from <IP_ADDRESS>:6444 in 5.000000 seconds.
Network timeout <IP_ADDRESS>:6444: No response from host.
No response from <IP_ADDRESS>:6444 in 6.000000 seconds.

And when enabling logging, I see the same output as https://github.com/mill1000/midea-ac-py/issues/141#issuecomment-2138146814

@fcastilloec Have you tried setting the "Maximum Connection Lifetime" option? Are you blocking the device from accessing the internet?

Sorry I've missed the question. No I am not blocking devices from internet access, and I don't think that there are any internet interruptions during that error time.

@mill1000 I've changed the "Maximum Connection Lifetime" option to 120 seconds and I still see the same errors. I'm not blocking any device from accessing the internet, and as @patrykwoj91 mentioned, there haven't been any internet interruptions during that time.
I actually just got another error while typing this response. It regularly gets disconnected and shown as "Unavailable" in Home Assistant due to these errors.

I'll need extended debug logs from both of you. About 10 minutes worth.
Ultimately there might be nothing I can do about it. I can't stop the device from disconnecting if it chooses to.

home-assistant_midea_ac_2024-06-28T20-31-45.095Z.log
Log from several hours.

Aside from the 1 error at the beginning this connection looked solid for several hours.

@fcastilloec Have you tried setting the "Maximum Connection Lifetime" option? Are you blocking the device from accessing the internet?

What should it be set to to try and mitigate these errors?

90 seconds is a common value, but you can determine the best value by looking at the logs and determining how often the device disconnects itself.

Closing this issue as it seems to have gone stale.
Feel free to re-open if you are still having issues.


PayPal Button not Rendering in IE11 (Intermittent)

I don't have steps to reproduce, nor a code example that I can provide, but we're seeing some strange, intermittent behaviour on IE11 and edge, involving postMessage coming out of checkout.js.
We're having a very hard time debugging from some of the error messages we get back from checkout.js.
Here's a screenshot:

We're using a fairly barebones set-up, we're using a script tag as outlined in the Usage notes, and calling window.paypal.Button.render to set up the PayPal button.
In the cases that the button is rendered, and the overlay is open, when the user selects "Continue", the overlay is closed, but there's no xcomponent_destroy called, as we've seen in other browsers (Chrome, Firefox, Safari), the lightbox iframe remains in the DOM:

I should also mention this is occurring in the sandbox environment only, for now. Here's the code we're using to setup the PayPal button.
function* createCallbackChannel(outerChannel) {
  const env = PAYPAL_ENVIRONMENT;
  const locale = yield select(selectPayPalLocaleCode);

  return eventChannel(emit => {
    // https://developer.paypal.com/docs/integration/direct/express-checkout/integration-jsv4/other-integrations/#in-context-with-form
    window.paypal.Button.render({
      env,
      locale,
      payment: (resolve, reject) => {
        outerChannel.take(({ token }) => {
          if (token) {
            return resolve(token);
          }
          return reject({ result: 'No valid token provided.' });
        });
        emit({
          type: PAYPAL.FETCH_TOKEN_CALLBACK,
        });
      },
      onAuthorize: data => {
        emit({
          type: PAYPAL.AUTHORIZATION_SUCCESS,
          payload: {
            token: data.paymentToken,
            payerID: data.payerID,
          },
        });
      },
      onCancel: () => {
        // noop
      },
      style: {
        color: 'gold',
        shape: 'rect',
        size: 'medium',
      },
    }, '#paypal-button');

    return () => {};
  });
}

Thanks for all of the details! When you say you're only seeing this issue on sandbox, do you mean you haven't yet tried in production mode, or that it works for you consistently in production mode?

Also, is the overlay visible on the page, or is it just the invisible iframe which isn't cleared up?

We've not yet tested in production mode. The overlay is no longer visible, just the invisible iframe not being cleared up. If we set display to block on the iframe, we see the PayPal semi-transparent black background and logo.

OK thanks, that's a really useful data point. I'll take another look tonight when I get back to my laptop.
Can you just quickly let me know which version of the script you see, when you type paypal.version?

Thanks again, i'll keep trying to drill down to the underlying issue. Reported version of PayPal version is 4.0.47.
Any chance we can either set the postRobot.CONFIG.ACK_TIMEOUT directly before calling render? Will that have an effect? Is there any chance we could manually call a paypal.postRobot.send?

We've recently found that setting a delay before rendering the button offers promising results. Our URL changes as we're opening a cart overlay, so we're thinking that if the URL doesn't match exactly when the button renders, there's a domain mismatch, postRobot fails with an ACK timeout and the button unloads itself. Does that sound like a possible scenario?

Hi again @stefannew. So I tried this case again in IE11, and I was still unable to reproduce the problem. I'm inspecting the dom, and I see all of the element cleared up as they're supposed to be.
Can you please try with this reference implementation, and tell me if you see the issue: http://www.bluesuncorp.co.uk/icv4/sandbox.htm -- that should narrow down if it's happening due to something in your integration, or if we have different browser settings, or something.

Any chance we can either set the postRobot.CONFIG.ACK_TIMEOUT directly before calling render?

I don't think the time-out itself is the problem here. If that timeout triggers after 1000ms, chances are the message will never go through.

We've recently found that setting a delay before rendering the button offers promising results. Our URL changes as we're opening a cart overlay, so we're thinking that if the URL doesn't match exactly when the button renders, there's a domain mismatch, postRobot fails with an ACK timeout and the button unloads itself. Does that sound like a possible scenario?

I'm interested to hear more details about this use-case. Does the domain change, or just the url path? If it's the domain, there may be something to this theory. When the popup window is first rendered, we send down the domain of the rendering page, and then it tries to message up to that specific domain only.

@bluepnume I've tried your suggested implementation a number of times, on virtual machines, and on my Windows machine and am unable to reproduce the issue, so I'd wager it's something to do with our implementation.
With regards to our use-case, the domain doesn't change, just the URL path. We've had some luck in delaying the URL change, so we're going to forge ahead with that and run some more tests.
I appreciate your help, and I'll get back with any additional information I have if any further tests yield any more revealing results. Still seems a little suspicious that it's only occurring on IE, but I feel it's kind of par for the course with this particular browser.
Thanks again.

OK, thanks for the info -- definitely narrows it down a bit.
When you change the url path -- is that with html5 pushState, or a hash change, or is it a full location change? And if it's the latter, how do you keep the button rendered -- is the page not re-rendered entirely?

We're using html5 pushState, from react-router - I guess the fallback is a hash change, but I'm pretty sure IE11 supports pushState. Hopefully the button is only rendered after the location changes.

Is there any chance the button iframe is being removed and re-added to the DOM?

I don't think the iframe is being re-rendered, but i'll run some tests on my end to make sure. Delaying the button doesn't seem to have any effect on the invisible iframe. That issue is also intermittent, doesn't seem to have any errors associated with it, either.

Can I also just check -- did you see the invisible iframe at all with the reference implementation at http://www.bluesuncorp.co.uk/icv4/sandbox.htm ?

I'll run some tests.
I have seen an issue sometimes in which the user cancels the transaction, the iframe is closed, but the lightbox is still present and no amount of interaction can remove it, save for a hard refresh:

Selecting "Continue" has no effect, and clicking the 'x' logs ppxo_xc_ppcheckout_focus, and in turn throws an error:
SCRIPT5022: Unspecified error.
Error: Unspecified error.
   at __BROWSERTOOLS_CONSOLE_SAFEFUNC (eval code:6:29)
   at print (https://www.paypalobjects.com/api/checkout.js:4910:21)
   at log (https://www.paypalobjects.com/api/checkout.js:5037:17)
   at info (https://www.paypalobjects.com/api/checkout.js:5068:13)
   at log (https://www.paypalobjects.com/api/checkout.js:4756:21)
   at Anonymous function (https://www.paypalobjects.com/api/checkout.js:9203:25)
   at _loop2 (https://www.paypalobjects.com/api/checkout.js:1077:25)
   at SyncPromise.prototype.dispatch (https://www.paypalobjects.com/api/checkout.js:1107:17)
   at SyncPromise.prototype.then (https://www.paypalobjects.com/api/checkout.js:1125:13)
   at syncPromiseTry (https://www.paypalobjects.com/api/checkout.js:1170:13)

IE version is 11.0.9:

Windows 7 VM.
Not sure if related?

Always fantastic when IE gives useful errors like "Unspecified Error". I also wasn't familiar with __BROWSERTOOLS_CONSOLE_SAFEFUNC but looks like this is being auto-added by the IE dev tools.
Elsewhere I have console logs in a try/catch because IE sometimes has a tendency to error out when logging, in some scenarios. I'll make sure this line is wrapped in a try/catch too, that ought to at least remove some of the noise here.

OK, published<EMAIL_ADDRESS>with catch-all for console.log errors. The next release of checkout.js will pick this up, I'll let you know here when that happens, probably early next week.
Still looking into reproducing other reported issues.

Hi @stefannew -- any new info on this one? Thanks!

@bluepnume Hey, sorry for the lapse in response, we've resolved the iframe issue in sort of a roundabout way, by removing any remaining iframes ourselves when the auth callback occurs and they're still present.
As for the button sometimes not rendering on IE11, we've had absolutely no luck debugging - there's a postRobot error (typical ACK timeout) and that's that. It sometimes happens, it sometimes doesn't... we're going to continue looking into it, but I haven't had a chance to get a version of the app up somewhere for you to try and repro apologies.
I'll keep you posted with any more information.


Debian-based Postgres

I've been playing around with creating a Debian-based Postgres image. This is mainly to have access to glibc for various extensions that I couldn't get to work nicely on Alpine.
Got it to a stage where I can run a cluster and verify that replication works as expected, and hoping someone here can help me check my approach.
Dockerfile: https://gist.github.com/GeoTob/8a9a2ee262e9fbf649e92c7f35f07593
I'm a bit worried that I've missed something critical, since kubedb dependencies may have been compiled against / tuned for Alpine. I'd like to be able to keep this up-to-date with future KubeDB versions and patches -- also happy to contribute my Dockerfile to the project if you're interested in rolling it into KubeDB/postgres

See: https://hub.docker.com/_/postgres
There are Debian and Alpine-based images

True, could've just grabbed the official Postgres-debian - thanks.
My question is more around installing the KubeDB scripts & binaries. Is the approach used in the Dockerfile OK ?


Config for app.env can't be retrieved on routes/api.php

In my apidoc.php, I set app.env to documentation
'config' => [
      'app.env' => 'documentation'
],

Then in routes/api.php I wanna do something like:
if (config('app.env') == 'documentation') {
    Route::get('/', 'API\UserController@get');
}

But config('app.env') still return local here, which is what i config in my .env. Is this a bug?

You sure your config isn't already cached? Try running php artisan config:clear.

Yes I'm sure my config isn't cached. I did try to clear config & route cache, but it won't help

I think the behavior is correct. config('app.env') means get the env key from app.php file in config folder which will return the value of this config/app.php line 29: 'env' => env('APP_ENV', 'production'),
So, using the sample code you provided above, the correct way to access the value is config('apidoc.config.app.env'). However, this will return null cos laravel uses . as separator so what laravel will actually look for is config/apidoc.php, an array config, an app key in the config array, then an env key in the app array, which doesn't exist.
In order to access that value, you have to rename the key to something without the dot. For example,
Rename the key from app.env to env so you can access the value with config('apidoc.config.env')

I'm sorry, I didn't read this issue well. What you're trying to achieve won't work. The config values in apidoc.php are only set when attempting to invoke a route to generate a response. What you're trying to do (conditional route declarations) is not a responsibility of this package. You can try what @djunehor suggested above.

Thanks for your help @djunehor @shalvah


Не рабочий конфиг

Ошибка:
sing-box run -c wg0.json
←[31mERROR←[0m[0000] outbound/wireguard[wireguard-out]: failed to read packet from tun device: invalid argument
←[31mERROR←[0m[0003] dns: lookup failed for github.com: exchange6: Post "https://dns.adguard-dns.com/dns-query": dial tcp <IP_ADDRESS>:443: connectex: An established connection was aborted by the software in your host machine. | dial tcp <IP_ADDRESS>:443: An invalid argument was supplied. | exchange4: Post "https://dns.adguard-dns.com/dns-query": dial tcp <IP_ADDRESS>:443: connectex: An established connection was aborted by the software in your host machine. | dial tcp <IP_ADDRESS>:443: An invalid argument was supplied.
←[31mFATAL←[0m[0003] start service: initialize rule-set[0]: initial rule-set: antizapret: Get "https://github.com/savely-krasovsky/antizapret-sing-box/releases/latest/download/antizapret.srs": exchange6: Post "https://dns.adguard-dns.com/dns-query": dial tcp <IP_ADDRESS>:443: connectex: An established connection was aborted by the software in your host machine. | dial tcp <IP_ADDRESS>:443: An invalid argument was supplied. | exchange4: Post "https://dns.adguard-dns.com/dns-query": dial tcp <IP_ADDRESS>:443: connectex: An established connection was aborted by the software in your host machine. | dial tcp <IP_ADDRESS>:443: An invalid argument was supplied.
Конфиг:
{
"log": {
"level": "warn"
},
"dns": {
"servers": [
{
"tag": "adguard-dns",
"address": "https://dns.adguard-dns.com/dns-query",
"address_resolver": "local-dns",
"detour": "wireguard-out"
},
{
"tag": "local-dns",
"address": "local",
"detour": "direct-out"
}
],
"rules": [
{
"outbound": "any",
"server": "local-dns"
}
]
},
"inbounds": [
{
"type": "tun",
"inet4_address": "<IP_ADDRESS>/30",
"auto_route": true,
"strict_route": true,
"sniff": true
}
],
"outbounds": [
{
"type": "direct",
"tag": "direct-out"
},
{
"type": "wireguard",
"tag": "wireguard-out",
"server": "<IP_ADDRESS>",
"server_port": 2408,
"system_interface": true,
"local_address": [
"<IP_ADDRESS>/32",
"2606:4700:110:843f:dc80:6f45:9df7:c43d/128"
],
"private_key": "4GpS7uCwST7RnnBfxNLpJCrPByonxGd+z8YLscSLk2w=",
"peer_public_key": "bmXOC+F1FxEMF9dyiK2H5/1SUtzH0JuVo51h2wPfgyo="
},
{
"type": "dns",
"tag": "dns-out"
}
],
"route": {
"rules": [
{
"rule_set": "antizapret",
"outbound": "wireguard-out"
},
{
"protocol": "dns",
"outbound": "dns-out"
}
],
"rule_set": [
{
"tag": "antizapret",
"type": "remote",
"format": "binary",
"url": "https://github.com/savely-krasovsky/antizapret-sing-box/releases/latest/download/antizapret.srs",
"download_detour": "wireguard-out"
}
],
"auto_detect_interface": true
},
"experimental": {
"cache_file": {
"enabled": true
}
}
}

Посмотрите конфиг тут: https://krasovs.ky/2024/08/05/sing-box-bypass.html
Я внес правки, если есть какие-то проблемы, оставляйте комменатрии там. Если всё ок, перенесу пример в этот реп.


Problem with FullPageJS

The text which is to modify in the section 1 works but the text which is in the section 2, 3, etc. does not work. It has an opacity of 0. Would you know how to help me?
$('#fullpage').fullpage({
    sectionsColor: [ '#f2f2f2', '#4BBFC3', '#7BAABE', 'whitesmoke', '#000' ],
    resize: true,
    autoScrolling: false,
    scrollBar: true,
    afterRender: function(){
      window.sr = new scrollReveal()
    }
});

Some issues have come up before with fullPage.js, see #45 #84 #14
This except from #45 in particular might be of some help:

From the fullPage.js FAQ:

fullPage.js doesn't actually scroll the site but it changes the top or translate3d property of the site. Only when using the fullPage.js option scrollBar:true or autoScrolling:false it will actually scroll the site in a way it is accessible for the scrollTop property.


Update SegmentList usage

Instead of explicitly calling out STIS and COS, you can use the generic SegmentList since no data is actually being coadded together here.
https://github.com/spacetelescope/ullyses/blob/main/ullyses/timeseries.py#L192

I believe the original intention here was to abstract away the instrument-specific usage of child SegmentList classes in timeseries. This may be possible, but would require testing. This ticket will be left open for now, but this is an enhancement to generalize the code, not a necessity.


graph evolution of a numeric value over time

[cross-posting https://github.com/githubocto/flat/discussions/15 in case this is a better place to discuss this]
🧙  The killer feature for me would be to be able to graph the evolution of values over time (i.e. across commits)
i.e. I'm using a Flat action to scrape the number of "things" from an API. I want to graph number of "things" over time 📈
Let me know if I can help!

hey @julien-c! Having visualizations of the data (especially over time/commits) is a great idea! It's a bit of a beast, and so we won't be able to tackle it as a team anytime soon, but I'll leave this issue open in case anyone is interested in taking a stab at it!

@julien-c you might want to try my Data Preview vscode extension for some basic pivots and plots. See features for more info.
Data preview supports loading data from public https data sources. So, you can use github actions to update the data files in your repo and then reload them in Data Preview in grid or basic charts view.
Just be aware there is currently an issue with the dark data preview UI theme on darwin OS. So, just change that setting to light.
Have fun!

This feature would be awesome, I was also looking for it. This is especially relevant for Flat Action that modifies the same file, not appends to it with a "datetime" column.
@RandomFractals I tried Data Preview and it does not show historic values - what @julien-c requested, but thanks for the suggestion.

@RandomFractals I tried Data Preview and it does not show historic values - what @julien-c requested, but thanks for the suggestion.

I only suggested Data Preview as an alternative data viewer in vscode that also allows you to crate basic charts. It doesn't track data files changes for historic values display, and I don't think I implied that in my comment.


Fixed byte-string issues for telnet libs due to migration from python…

…2 to python3
Signed-off-by: Orest Kreminskyi<EMAIL_ADDRESS>

This change is

@rbbratta @esmacgil @seungweonpark @lijenni
Could you please review these changes for byte-sting issues in telnet libs modules?

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

taf/testlib/cli_template.py, line 323 at r1 (raw file):

        if self.prompt:
            for single_prompt in self.prompt if isinstance(self.prompt, list) else [self.prompt]:
                str_prompt = single_prompt.decode()

don't use bare decode, always specify the encoding.
see OpenStack Python3 guidelines
https://wiki.openstack.org/wiki/Python3#bytes.decode_and_unicode.encode
bytes.decode and unicode.encode
Python has a notion of "default encoding": sys.getdefaultencoding(). On Python 2, the default encoding is ASCII, whereas it is UTF-8 on Python 3.
Don't write data.decode() or text.encode() without parameter, because you will use a different encoding on Python 2 and Python 3.
Use an explicit encoding instead. Example: data.decode('utf-8') or text.encode('utf-8'). The right encoding depends on the use case, but UTF-8 is usually a good candidate (it is a superset of ASCII).

taf/testlib/clitelnet.py, line 75 at r1 (raw file):

        self.prompt = prompt.encode()
        self.pass_prompt = pass_prompt.encode()
        self.login_prompt = login_prompt.encode()

same, use explicit encoding,  utf-8

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

taf/testlib/cli_template.py, line 323 at r1 (raw file):
Previously, rbbratta (Ross Brattain) wrote…
don't use bare decode, always specify the encoding.
see OpenStack Python3 guidelines
https://wiki.openstack.org/wiki/Python3#bytes.decode_and_unicode.encode
bytes.decode and unicode.encode
Python has a notion of "default encoding": sys.getdefaultencoding(). On Python 2, the default encoding is ASCII, whereas it is UTF-8 on Python 3.
Don't write data.decode() or text.encode() without parameter, because you will use a different encoding on Python 2 and Python 3.
Use an explicit encoding instead. Example: data.decode('utf-8') or text.encode('utf-8'). The right encoding depends on the use case, but UTF-8 is usually a good candidate (it is a superset of ASCII).

Thanks for your comment. New patch set has been uploaded.

taf/testlib/clitelnet.py, line 75 at r1 (raw file):
Previously, rbbratta (Ross Brattain) wrote…
same, use explicit encoding,  utf-8

Done.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable


Switch to a new command 'check'

This will make way for an optional command 'remote' or 'local' to analyze the dependencies of a remote repository or a local project.

Check why poetry cannot be determined if POETRY_VERSION isn't specified. Suggest site to test on is M extranet

Stop requirements.txt been left over when running on a local repo.